Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/exception/exception-to-review-message.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const exceptionToReviewMessage = {
'MODULE_NOT_FOUND': 'sepertinya ada file yang tidak terimport dengan baik, pastikan path yang kamu gunakan sudah benar atau pastikan tidak ada typo pada path yang kamu masukkan.',
'FAIL_INSTALLING_PACKAGE': 'package pada aplikasi yang kamu buat belum bisa di-install dengan baik.',
'PROJECT_CONTAIN_DATABASE_DEPENDENCY': 'file package.json yang kamu buat memiliki dependency database yang tidak diperbolehkan. Perlu diingat bahwa untuk mengerjakan submission ini kamu tidak dipebolehkan menggunakan database seperti Postgres, MSSQL, MySQL, MariaDB, atau MongooDB untuk menyimpan data. Silakan <b>hapus dependency<b> tersebut dari file package.json ya.',
'PROJECT_CONTAIN_OTHER_FRAMEWORK_DEPENDENCY': 'file package.json yang kamu buat memiliki dependency Framework lain yang tidak diperbolehkan. Perlu diingat bahwa untuk mengerjakan submission ini kamu tidak dipebolehkan menggunakan Framework lain selain Hapi Framework. Silakan <b>hapus dependency<b> tersebut dari file package.json ya.',
'PROJECT_CONTAIN_OTHER_FRAMEWORK_DEPENDENCY': 'file package.json yang kamu buat memiliki dependency Framework lain yang tidak diperbolehkan. Perlu diingat bahwa untuk mengerjakan submission ini kamu tidak dipebolehkan menggunakan Framework lain selain Hapi dan Express Framework. Silakan <b>hapus dependency<b> tersebut dari file package.json ya.',
'PROJECT_CONTAIN_NODE_MODULES' : 'project yang kamu buat memiliki folder <strong>node_modules</strong>. Perlu diingat bahwa <strong>node_modules</strong> tidak perlu dikirimkan ke dalam submission, <strong>node_modules</strong> akan didapatkan dengan cara menajalankan perintah install pada project. Silakan hapus terlebih dahulu folder <strong>node_modules</strong> seperti ketentuan submission.<img src="https://dicoding-web-img.sgp1.cdn.digitaloceanspaces.com/original/academy/dos:3fcde7eec6afcdee10efb449bd0f6dca20230215121207.png" style="width: 756px;" class="fr-fic fr-dib fr-draggable" data-success="true">',
'RUNNER_COMMAND_CONTAIN_NODEMON' : 'kamu menggunakan nodemon pada start script. Pada submission ini kamu tidak diperbolehkan menggunakan nodemon sebagai start script utama, jika kamu ingin menggunakan nodemon silakan simpan di dalam start script lain, contohnya<pre>{<br>&nbsp;&nbsp;&nbsp;&nbsp;"name": "submission",<br>&nbsp;&nbsp;&nbsp;&nbsp;...<br>&nbsp;&nbsp;&nbsp;&nbsp;"scripts": {<br>&nbsp;&nbsp;&nbsp;&nbsp;"start": "node src/server.js",<br><strong>&nbsp; &nbsp; &nbsp; &nbsp; "start-dev": "nodemon src/server.js",</strong><br>&nbsp;&nbsp;&nbsp;&nbsp;}<br>}</pre><br><img src="https://dicoding-web-img.sgp1.cdn.digitaloceanspaces.com/original/academy/dos:8d32e710e75d96597c17482ead576b4d20230215121505.png" style="width: 732px;" class="fr-fic fr-dib fr-draggable" data-success="true">'
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export default class SubmissionProjectFactory {
throw new ProjectErrorException('PROJECT_CONTAIN_DATABASE_DEPENDENCY')
}

const possibleOtherFrameworkDependencies = ['express']
const possibleOtherFrameworkDependencies = ['@nestjs/core', 'koa']
const isProjectContainOtherFramework = dependencies.some(dependency => possibleOtherFrameworkDependencies.includes(dependency))
if (isProjectContainOtherFramework) {
throw new ProjectErrorException('PROJECT_CONTAIN_OTHER_FRAMEWORK_DEPENDENCY')
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ describe('create submission project test', () => {
});

it('should throw error when project contain other framework', function () {
mockFS.readFileSync.mockReturnValue('{ "scripts": { }, "dependencies": { "express": 10.2 }}')
mockFS.readFileSync.mockReturnValue('{ "scripts": { }, "dependencies": { "koa": 3 }}')

expect(() => submissionProjectFactory.create(submissionRequirement,'.'))
.toThrow(new ProjectErrorException('PROJECT_CONTAIN_OTHER_FRAMEWORK_DEPENDENCY'))
Expand Down
3 changes: 2 additions & 1 deletion src/service/report-generator/report-generator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,10 @@
raiseDomainEvent('report generated')
}

private getDraftDecision(isApproved: boolean, autoReviewConfig: any): boolean {

Check warning on line 44 in src/service/report-generator/report-generator.ts

View workflow job for this annotation

GitHub Actions / lint (14.x)

Unexpected any. Specify a different type
const allowedCoursesThatFullyGrading = [
342, // Back-End Pemula with Google Cloud
342, // Back-End Pemula with Google Cloud
261, // Back-End Pemula with AWS
]

if (allowedCoursesThatFullyGrading.includes(autoReviewConfig.course_id)) {
Expand All @@ -60,7 +61,7 @@

}

getReviewMessageWithTemplate(reviewResult: ReviewResult, autoReviewConfig: any) {

Check warning on line 64 in src/service/report-generator/report-generator.ts

View workflow job for this annotation

GitHub Actions / lint (14.x)

Unexpected any. Specify a different type
const mainTemplate = templates.find(template => template.courseId === autoReviewConfig?.course_id)

if (!mainTemplate) {
Expand All @@ -80,7 +81,7 @@
}


private getAutoReviewConfig(projectPath: string): any | null {

Check warning on line 84 in src/service/report-generator/report-generator.ts

View workflow job for this annotation

GitHub Actions / lint (14.x)

Unexpected any. Specify a different type

const configFilePath = `${projectPath}/auto-review-config.json`
if (!existsSync(configFilePath)) {
Expand Down
Loading