Merge branch 'code-differently:main' into Shawn-lesson16

Author: Djtheman1

.github/workflows/check_lesson_17_java_pr.yaml

@@ -4,7 +4,7 @@ on:
   pull_request:
     branches: [ "main" ]
     paths:
-      - "lesson_17/oop/**"
+      - "lesson_17/bank/**"
 
 jobs:
   build:
@@ -24,5 +24,5 @@ jobs:
         distribution: 'temurin'
 
     - name: Build Lesson 17 with Java
-      working-directory: ./lesson_17/oop
+      working-directory: ./lesson_17/bank
       run: ./gradlew check

.github/workflows/check_push.yml

@@ -18,6 +18,9 @@ on:
       - "lesson_13/maps_java/**"
       - "lesson_13/maps_ts/**"
       - "lesson_14/exceptions/**"
+      - "lesson_15/tdd/**"
+      - "lesson_16/objects/**"
+      - "lesson_17/bank/**"
 
 jobs:
   build:
@@ -125,4 +128,14 @@ jobs:
 
     - name: Build Lesson 15 with Java
       working-directory: ./lesson_15/tdd
+      run: |
+        ./gradlew assemble
+        ./gradlew spotlessCheck
+
+    - name: Build Lesson 16 with Java
+      working-directory: ./lesson_16/objects
+      run: ./gradlew check
+
+    - name: Build Lesson 17 with Java
+      working-directory: ./lesson_17/bank
       run: ./gradlew check

lesson_10/libraries/package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "start": "node build/lesson10.js",
     "dev": "tsc-watch --noClear -p ./tsconfig.build.json --onSuccess \"node ./build/lesson10.js\"",
-    "test": "node --experimental-vm-modules node_modules/.bin/jest",
+    "test": "node --experimental-vm-modules node_modules/.bin/jest --coverage",
     "compile": "tsc -p tsconfig.build.json",
     "prepare": "npm run compile",
     "pretest": "npm run compile",
@@ -42,4 +42,4 @@
     "csv-parser": "^3.0.0",
     "uuid": "^10.0.0"
   }
-}
+}

lesson_16/objects/gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.0.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.10.2-bin.zip
 networkTimeout=10000
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

lesson_16/objects/objects_app/build.gradle.kts

@@ -2,6 +2,8 @@ plugins {
     // Apply the application plugin to add support for building a CLI application in Java.
     application
     eclipse
+    jacoco
+    id("io.freefair.lombok") version "8.10.2"
     id("com.diffplug.spotless") version "6.25.0"
 	  id("org.springframework.boot") version "3.2.2"
     id("com.adarshr.test-logger") version "4.0.0"
@@ -16,17 +18,14 @@ repositories {
 
 dependencies {
     // Use JUnit Jupiter for testing.
-    testImplementation("com.codedifferently.instructional:instructional-lib")
     testImplementation("org.junit.jupiter:junit-jupiter:5.9.1")
     testImplementation("org.springframework.boot:spring-boot-starter-test")
     testImplementation("org.assertj:assertj-core:3.25.1")
     testImplementation("at.favre.lib:bcrypt:0.10.2")
 
     // This dependency is used by the application.
-    implementation("com.codedifferently.instructional:instructional-lib")
     implementation("com.google.guava:guava:31.1-jre")
     implementation("com.google.code.gson:gson:2.10.1")
-    implementation("org.projectlombok:lombok:1.18.30")
     implementation("org.springframework.boot:spring-boot-starter")
 }
 
@@ -38,8 +37,29 @@ application {
 tasks.named<Test>("test") {
     // Use JUnit Platform for unit tests.
     useJUnitPlatform()
+    finalizedBy(tasks.jacocoTestReport)
 }
 
+tasks.jacocoTestReport {
+  dependsOn(tasks.test)
+  reports {
+    xml.required = true
+  }
+}
+
+tasks.jacocoTestCoverageVerification {
+  violationRules {
+    rule {
+      limit {
+        minimum = "0.8".toBigDecimal()
+      }
+    }
+  }
+}
+
+tasks.check {
+	dependsOn(tasks.jacocoTestCoverageVerification)
+}
 
 configure<com.diffplug.gradle.spotless.SpotlessExtension> {
 

lesson_16/objects/objects_app/src/test/java/com/codedifferently/lesson16/Lesson16Test.java

@@ -3,11 +3,13 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import org.junit.jupiter.api.Test;
+import org.springframework.boot.test.context.SpringBootTest;
 
+@SpringBootTest(useMainMethod = SpringBootTest.UseMainMethod.WHEN_AVAILABLE)
 class Lesson16Test {
 
   @Test
-  void testCanVote() {
+  void testMain() {
     assertThat(new Lesson16()).isNotNull();
   }
 }

lesson_16/objects/settings.gradle.kts

@@ -7,7 +7,5 @@
  * in the user manual at https://docs.gradle.org/8.0.2/userguide/multi_project_builds.html
  */
 
-includeBuild("../../lib/java/codedifferently-instructional")
-
 rootProject.name = "lesson_16"
 include("objects_app")

lesson_17/README.md

@@ -16,34 +16,27 @@ Please review the following resources before lecture:
 
 ## Homework
 
-- [ ] Complete [Library Management System](#library-management-system) assignment.
+- [ ] Complete [Applying SOLID principles](#applying-solid-principles-bank-atm) exercise.
+- [ ] Review [OOP Project](/project_oop/) documentation and complete user stories.
 
-## Library Management System
+## Applying SOLID Principles (Bank ATM)
 
-For this assignment, you will build a library management system using object oriented principles. Pay attention to the requirements careful in order to implement your system successfully.
+Your task for this assignment is add enhancements to an ATM simulator. The [BankAtm][bankatm-file] is at the center of the model, allowing us to add one or more `CheckingAccount` instances and make withdrawals or deposits via cash or check. You will need to implement at least two of the following functional enhancements to the `BankAtm` class WITHOUT adding a new method. Note that you can update existing methods, however.
 
 ### Functional Requirements
 
-* We want to define a book and keep track of its title, isbn, author(s), number of pages, and whether it is checked out or not.
-* We want define a patron with properties including their name and the books they have checked out.
-* We want to define a library that:
-    * Allows us to add/remove a book to and from its collection of books. 
-    * Allows us to register new patrons
-    * Allows us to check out books to patrons.
-    * Allows us to return books from patrons.
+* We want to support a `SavingsAccount` that works just like the `CheckingAccount`, but doesn't allow you to write checks against the account.
+* We want the `BankAtm` class to support the concept of a `BusinessCheckingAccount`. A business account requires that at least one of the owning accounts is a business.
+* In addition to supporting checks and cash, we also want to support the concept of another monetary instrument called a `MoneyOrder`. Unlike a `Check`, a `MoneyOrder` withdraws funds from a source account immediately on creation for the purposes of this simulation..
+* For traceability, all of the transactions in the `BankAtm` class should logged. Create an `AuditLog` class that keeps a record of all debits and credits to any account and integrate it with the `BankAtm` class. 
+* For the `depositFunds` method that accepts a cash amount, we'd like the ability to deposit funds in a variety of currencies. Add a parameter that accepts a currency type and a new object that encapsulates the currency converter logic for converting a cash amount to the account currency type.
 
 ### Technical Requirements
 
-* All of your types must be created in a unique sub-package under the `com.codedifferently.lesson17` namespace.
-* Include tests for all of your classes and public methods.
-* Include appropriate `javadoc` for your classes and methods. In VS Code, you can usually start a comment with `/**` and press `Enter` to let the IDE automatically create some starter text for you.
+* You must integrate new features into the `BankAtm` without adding a new public method. Existing public methods may be modified without breaking existing functionality.
+* You must update the `BankAtm` tests and may modify or add other applicable tests.
+* Feel free to add the minimal number of classes, interfaces, or abstract classes needed to fulfill each requirement.
+* You must update existing javadocs and may add new documentation for new types and methods you introduce.
 
-### Tips
-
-* Read the functional requirements carefully, and take special care to identify *objects*, their *data*, and their *actions*.
-* It is highly recommended that you write your tests firsts, and then build what you need to make the tests work. This is essential to [Test-Driven Development][tdd-article].
-* How will you ensure that books are properly checked out using the library? What if someone tries to check out a book the library doesn't own (or vice-versa with returns)?
-* What are the public methods that you need to have to get the system working? Don't make anything public that you don't need to fulfill the requirements of the assignment. For instance, adding a getter method for a book's ISBN makes sense, but adding a method for returning the library's hours of operation is beyond the scope of this assignment.
-* How can you use collections to manage books and patrons? For instance, it would be wise to ensure that patrons can only be registered once based on some unique identifier.
-
-[tdd-article]: https://semaphoreci.com/blog/test-driven-development
+[bank-folder]: ./bank/
+[bankatm-file]: ./bank/bank_app/src/main/java/com/codedifferently/lesson17/bank/BankAtm.java