Update assistest to suitable for grading submission that use ESLint > 9

Dockerfile

@@ -1,14 +1,17 @@
 FROM node:18-slim
 
-RUN mkdir /student-app && chown -R node:node /student-app
-RUN mkdir /report && chown -R node:node /report
-RUN usermod -aG root node
+RUN apt-get update && apt-get install -y gosu
 
-WORKDIR /app
-RUN chown -R node:node /app
-USER node
+RUN useradd -ms /bin/bash assistest
 
-COPY --chown=node:node . .
+RUN mkdir /home/assistest/app && chown -R assistest:assistest /home/assistest/app
+RUN mkdir /home/assistest/student-app && chown -R assistest:assistest /home/assistest/student-app
+RUN mkdir /home/assistest/report && chown -R assistest:assistest /home/assistest/report
+
+WORKDIR /home/assistest/app
+COPY --chown=assistest:assistest . .
+
+RUN npm config set package-lock false
 RUN yarn install --production=true
 
-ENTRYPOINT ["sh", "entrypoint.sh"]
+ENTRYPOINT ["/bin/bash", "entrypoint.sh"]

entrypoint.sh

@@ -1,3 +1,4 @@
 #!/bin/bash
-rm -f /reports/report.json || true
-"$@"
+rm -f /home/assistest/reports/report.json || true
+chown -R assistest:assistest /home/assistest/student-app
+exec gosu assistest "$@"

package.json

@@ -17,6 +17,7 @@
   "dependencies": {
     "@types/node": "^18.8.4",
     "axios": "^0.27.2",
+    "del": "^6.1.1",
     "dotenv": "^16.0.3",
     "minimist": "^1.2.7",
     "newman": "^5.3.2",
@@ -27,6 +28,7 @@
   },
   "devDependencies": {
     "@types/jest": "^29.1.2",
+    "@types/minimist": "^1.2.4",
     "@types/newman": "^5.3.1",
     "@typescript-eslint/eslint-plugin": "^5.40.0",
     "@typescript-eslint/parser": "^5.40.0",

src/entities/review-result/course-submission-review/course-submission-review.ts

@@ -39,7 +39,8 @@ class CourseSubmissionReview {
             rating: courseSubmissionAcception.rating,
             message: courseSubmissionAcception.messages,
             status: ReviewResultStatus.Approve,
-            checklist: this.submissionCriteriaCheck.reviewChecklistResult
+            checklist: this.submissionCriteriaCheck.reviewChecklistResult,
+            draft: false,
         }
     }
 
@@ -52,9 +53,10 @@ class CourseSubmissionReview {
             rating: 0,
             message: courseSubmissionRejection.messages,
             status: ReviewResultStatus.Reject,
-            checklist: this.submissionCriteriaCheck.reviewChecklistResult
+            checklist: this.submissionCriteriaCheck.reviewChecklistResult,
+            draft: true,
         }
     }
 }
 
-export default CourseSubmissionReview
+export default CourseSubmissionReview

src/entities/review-result/course-submission-review/review-result.ts

@@ -4,12 +4,13 @@ interface ReviewResult {
     rating: number,
     message: string,
     status: ReviewResultStatus,
-    checklist: SubmissionRequirement
+    checklist: SubmissionRequirement,
+    draft: boolean,
 }
 
 export enum ReviewResultStatus {
     Approve = "Approve",
     Reject = "Reject",
 }
 
-export default ReviewResult
+export default ReviewResult

src/interface/cli.ts

@@ -3,6 +3,7 @@ import {readdirSync} from "fs";
 import ReportGenerator from "../service/report-generator/report-generator";
 import * as path from "path";
 import * as minimist from 'minimist';
+import {sync} from "del";
 
 class Cli {
     private reportPath: string;
@@ -30,6 +31,8 @@ class Cli {
             const submissionPath = path.resolve(this.folderPath)
             const reviewResult = await main.reviewSubmission(submissionPath)
             reportGenerator.generate(reviewResult, submissionPath)
+            // remove all files in submissionPath except report.json
+            sync([`${submissionPath}/**`, `!${submissionPath}/report.json`], { force: true })
         }
     }
 

src/interface/implementation-test.ts

@@ -1,17 +1,18 @@
 import {readdirSync} from "fs";
-import * as path from "path";
+import {join} from "path";
 import {main} from "../service-provider";
 import ReportGenerator from "../service/report-generator/report-generator";
 
-const reportGenerator = new ReportGenerator('./report')
 async function run() {
-    const allSubmission = readdirSync('../experiment-storage/project')
+    const experimentStorageProjectPath = join(process.cwd(), 'experiment-storage', 'project')
+    const allSubmission = readdirSync(experimentStorageProjectPath)
 
     for (const submission of allSubmission) {
         console.log(`checking ${submission}`)
-        const submissionPath = path.resolve('../experiment-storage/project', submission)
+        const submissionPath = join(experimentStorageProjectPath, submission)
         const reviewResult = await main.reviewSubmission(submissionPath)
-        reportGenerator.generate(reviewResult, submission)
+        const reportGenerator = new ReportGenerator(submissionPath)
+        reportGenerator.generate(reviewResult, submissionPath)
     }
 }
 

src/service/eslint-checker/eslint-checker.ts

@@ -11,10 +11,10 @@ class EslintChecker {
 
         try {
             //check eslint config available or not
-            execSync('npx eslint ./', {
+            execSync('npx eslint ./ --ignore-pattern \'eslint.config.*\'', {
                 cwd: submissionProject.packageJsonPath, stdio: "pipe"
             })
-            const result = execSync('npx eslint ./ --rule \'linebreak-style:off\'', {
+            const result = execSync('npx eslint ./ --rule \'linebreak-style:off\' --ignore-pattern \'eslint.config.*\'', {
                 cwd: submissionProject.packageJsonPath,
                 stdio: "pipe"
             })

src/service/report-generator/report-generator.test.ts

@@ -3,6 +3,21 @@
 import ReviewResult, {ReviewResultStatus} from "../../entities/review-result/course-submission-review/review-result";
 import * as fs from "fs";
 
+function itShouldMeetAGRSReportSpec(report: any): void {
+    expect(report.submission_id).toBeDefined()
+    expect(typeof report.submission_id).toEqual('number')
+    expect(report.message).toBeDefined()
+    expect(typeof report.message).toEqual('string')
+    expect(report.rating).toBeDefined()
+    expect(typeof report.rating).toEqual('number')
+    expect(report.is_passed).toBeDefined()
+    expect(typeof report.is_passed).toEqual('boolean')
+    expect(report.is_draft).toBeDefined()
+    expect(typeof report.is_draft).toEqual('boolean')
+    expect(report.checklist_keys).toBeDefined()
+    expect(Array.isArray(report.checklist_keys)).toEqual(true)
+}
+
 describe('checklist id resolver test', () => {
     const reportGenerator = new ReportGenerator('./test/student/review-result/')
 
@@ -21,14 +36,17 @@
             rating: 5,
             status: ReviewResultStatus.Approve,
             checklist: submissionRequirement,
-            message: 'Selamat'
+            message: 'Selamat',
+            draft: false,
         }
 
         const studentProjectPath = './test/student-project/project-with-correct-autoreview-config'
         reportGenerator.generate(reviewResult, studentProjectPath)
 
         const result = JSON.parse(fs.readFileSync('./test/student/review-result/report.json').toString())[0]
 
+        itShouldMeetAGRSReportSpec(result)
+
         expect(result.checklist_keys).toEqual([
             "project_have_correct_port",
             "project_have_correct_runner_script",
@@ -38,7 +56,7 @@
             "api_can_update_book",
             "api_can_delete_book"
         ])
-        expect(result.message).toStrictEqual('<p>Hallo <strong>snder12</strong>, terima kasih telah sabar menunggu. Kami membutuhkan waktu untuk bisa memberikan <em>feedback</em> sekomprehensif mungkin kepada setiap peserta kelas. Dalam kesempatan ini ada &nbsp;4 (empat) hal yang ingin kami sampaikan.&nbsp;</p><p><strong>Pertama</strong>, kami ingin mengucapkan selamat! Karena kamu telah menyelesaikan tugas submission dari kelas Belajar Membuat Aplikasi Back-End untuk Pemula. Jangan lihat bintang yang kamu raih, tapi lihat kemajuan yang sudah kamu capai. Ingat semua <em>expert&nbsp;</em>dahulu pemula.&nbsp;</p><p><strong>K</strong><strong>edua</strong>, kamu boleh bangga karena telah menyelesaikan submission sesuai dengan kriteria yang telah kami tentukan. Mumpung masih hangat semangatnya langsung lanjut kelas selanjutnya yaitu <a href="https://www.dicoding.com/academies/266">Architecting on AWS (Membangun Arsitektur AWS di Cloud)</a> atau <a href="https://www.dicoding.com/academies/271">Belajar Fundamental Aplikasi Back-End</a>.&nbsp;</p><p><strong>Ketiga</strong>, beberapa lulusan tidak tahu mereka memiliki akses kelas selamanya. Sebagai informasi kelas Dicoding selalu <em>update&nbsp;</em>sehingga memiliki perbedaan minimal 30% dari sejak kelas dirilis. Silakan mampir kembali untuk melihat materi saat kamu membutuhkan <em>update</em>.&nbsp;</p><p><strong>K</strong><strong>eempat</strong>, karena sudah praktik langsung maka kamu sudah menguasai ilmu kelas dasar ini antara 75-90%. Salah satu cara agar meningkatkan penguasaan ilmu agar bisa lebih maksimal (&gt;90%) adalah dengan memperbanyak latihan atau mengajarkan ilmu kepada orang lain.</p><p>Salah satu misi Dicoding adalah menyebarkan ilmu yang bermanfaat. Kami berusaha membangun kurikulum standar global dengan harapan agar developer Indonesia bisa menjadi jawara di negeri sendiri. Namun misi ini tidak akan tercapai tanpa kolaborasi dari kita semua.</p><hr><p>Supaya aplikasimu menjadi lebih baik lagi, berikut <em>beberapa</em> <em>catatan</em> terkait submission kamu:</p><ul>Selamat</ul><hr><p>Silakan berkunjung ke <a href="https://www.dicoding.com/academies/261/discussions">academy discussion</a> untuk mengasah penguasaan ilmu kamu dan membuat ilmu yang kamu dapatkan bisa semakin berkah dan bermanfaat dengan membantu kawan-kawan kita yang saat ini masih berjuang.</p><p>Terima kasih telah membantu misi kami. Kesuksesan developer Indonesia adalah energi bagi kami. Jika memiliki pertanyaan atau saran terkait kelas, silakan email ke <a href="mailto:%[email protected]" rel="noreferrer noopener" target="_blank">[email protected]</a>.</p><hr><p style="text-align:right;"><em>Salam</em></p><p style="text-align:right;"><span style="color:rgb(226,80,65);">Dicoding Reviewer</span></p>')
+        expect(result.message).toStrictEqual('<p>Hallo <strong>snder12</strong>, terima kasih telah sabar menunggu. Kami membutuhkan waktu untuk bisa memberikan <em>feedback</em> sekomprehensif mungkin kepada setiap peserta kelas. Dalam kesempatan ini ada &nbsp;4 (empat) hal yang ingin kami sampaikan.&nbsp;</p><p><strong>Pertama</strong>, kami ingin mengucapkan selamat! Karena kamu telah menyelesaikan tugas submission dari kelas Belajar Membuat Aplikasi Back-End untuk Pemula. Jangan lihat bintang yang kamu raih, tapi lihat kemajuan yang sudah kamu capai. Ingat semua <em>expert&nbsp;</em>dahulu pemula.&nbsp;</p><p><strong>K</strong><strong>edua</strong>, kamu boleh bangga karena telah menyelesaikan submission sesuai dengan kriteria yang telah kami tentukan. Mumpung masih hangat semangatnya langsung lanjut kelas selanjutnya yaitu <a href="https://www.dicoding.com/academies/266">Architecting on AWS (Membangun Arsitektur AWS di Cloud)</a> atau <a href="https://www.dicoding.com/academies/271">Belajar Fundamental Aplikasi Back-End</a>.&nbsp;</p><p><strong>Ketiga</strong>, beberapa lulusan tidak tahu mereka memiliki akses kelas selamanya. Sebagai informasi kelas Dicoding selalu <em>update&nbsp;</em>sehingga memiliki perbedaan minimal 30% dari sejak kelas dirilis. Silakan mampir kembali untuk melihat materi saat kamu membutuhkan <em>update</em>.&nbsp;</p><p><strong>K</strong><strong>eempat</strong>, karena sudah praktik langsung maka kamu sudah menguasai ilmu kelas dasar ini antara 75-90%. Salah satu cara agar meningkatkan penguasaan ilmu agar bisa lebih maksimal (&gt;90%) adalah dengan memperbanyak latihan atau mengajarkan ilmu kepada orang lain.</p><p>Salah satu misi Dicoding adalah menyebarkan ilmu yang bermanfaat. Kami berusaha membangun kurikulum standar global dengan harapan agar developer Indonesia bisa menjadi jawara di negeri sendiri. Namun misi ini tidak akan tercapai tanpa kolaborasi dari kita semua.</p><hr><p>Supaya aplikasimu menjadi lebih baik lagi, berikut <em>beberapa</em> <em>catatan</em> terkait submission kamu:</p><ul>Selamat</ul><hr><p dir="ltr">Silakan berkunjung ke <a href="https://www.dicoding.com/academies/261/discussions" rel="noopener noreferrer" target="_blank">forum diskusi</a> untuk mengasah kembali penguasaan ilmu kamu dan membuat ilmu kamu bisa semakin bermanfaat dengan membantu developer yang lain.&nbsp;</p><p dir="ltr">Terima kasih telah membantu misi kami. Kesuksesan developer Indonesia adalah energi bagi kami. Jika memiliki pertanyaan terkait hasil submission, silakan mengikuti prosedur&nbsp;<a href="https://help.dicoding.com/academy-dicoding/prosedur-banding-hasil-review-submission-kelas" rel="noopener noreferrer" target="_blank">berikut</a>.</p><hr><p style="text-align: right;"><em>Salam</em></p><p style="text-align: right;"><span style="color: rgb(226, 80, 65);">Dicoding Reviewer</span></p>')
     });
 })
 

src/service/report-generator/report-generator.ts

@@ -3,6 +3,7 @@
 import {existsSync, readFileSync, writeFileSync} from "fs";
 import * as templates from "../../config/review-template.json";
 import raiseDomainEvent from "../../common/domain-event";
+import {join} from "path";
 
 class ReportGenerator {
     private readonly reportPath: string;
@@ -25,25 +26,41 @@
             message: this.getReviewMessageWithTemplate(reviewResult, autoReviewConfig),
             submission_path: submissionPath,
             checklist: reviewResult.checklist,
-            checklist_keys: this.getCompletedChecklist(reviewResult)
+            checklist_keys: this.getCompletedChecklist(reviewResult),
+            is_passed: isApproved,
+            is_draft: this.getDraftDecision(isApproved, autoReviewConfig),
         };
 
         this.result.push(summary);
         fs.mkdirSync(this.reportPath, {recursive: true});
+        const fileReportPath = join(this.reportPath, 'report.json')
 
-        writeFileSync(`${this.reportPath}/report.json`, JSON.stringify(this.result), {
+        writeFileSync(fileReportPath, JSON.stringify(this.result), {
             mode: '0664'
         })
         raiseDomainEvent('report generated')
     }
 
+    private getDraftDecision(isApproved: boolean, autoReviewConfig: any): boolean {
+        const allowedCoursesThatFullyGrading = [
+          342, // Back-End Pemula with Google Cloud
+        ]
+
+        if (allowedCoursesThatFullyGrading.includes(autoReviewConfig.course_id)) {
+            // set `draft` to false, if approved is true
+            return !isApproved
+        }
+
+        return true
+    }
+
     private getCompletedChecklist(reviewResult: ReviewResult) {
         return Object.keys(reviewResult.checklist)
             .filter(requirementName => reviewResult.checklist[requirementName].status)
 
     }
 
-    getReviewMessageWithTemplate(reviewResult: ReviewResult, autoReviewConfig) {
+    getReviewMessageWithTemplate(reviewResult: ReviewResult, autoReviewConfig: any) {
         const mainTemplate = templates.find(template => template.courseId === autoReviewConfig?.course_id)
 
         if (!mainTemplate) {
@@ -63,7 +80,7 @@
     }
 
 
     private getAutoReviewConfig(projectPath: string): any | null {
 
         const configFilePath = `${projectPath}/auto-review-config.json`
         if (!existsSync(configFilePath)) {