feat: schemas,controllers for the practice section #4967
Conversation
…stem from the moderators pov
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
|
Code Climate has analyzed commit 5a08114 and detected 0 issues on this pull request. View more on Code Climate. |
| @@ -0,0 +1,4 @@ | |||
| # frozen_string_literal: true | |||
There was a problem hiding this comment.
Difficulty level has been moved to enum. So this will not required anymore.
| @@ -0,0 +1,5 @@ | |||
| # frozen_string_literal: true | |||
|
|
|||
| class Category < ApplicationRecord | |||
There was a problem hiding this comment.
It wil be better to rename the model to QuestionCategory
| @@ -0,0 +1,9 @@ | |||
| # frozen_string_literal: true | |||
|
|
|||
| class DifficultyLevel < ApplicationRecord | |||
There was a problem hiding this comment.
It will be better to rename the model to QuestionDifficultyLevel
| @@ -92,7 +92,7 @@ | |||
|
|
|||
| create_table "assignments", force: :cascade do |t| | |||
| t.string "name" | |||
| t.datetime "deadline", null: false | |||
| t.datetime "deadline", precision: nil, null: false | |||
There was a problem hiding this comment.
Rollback this change, if not required
| @@ -451,6 +477,9 @@ | |||
| t.string "educational_institute" | |||
| t.boolean "subscribed", default: true | |||
| t.string "locale" | |||
| t.jsonb "submission_history", default: [], array: true | |||
| t.boolean "public", default: true | |||
There was a problem hiding this comment.
Is this added public field required ?
|
|
||
| FactoryBot.define do | ||
| factory :category do | ||
| name { "MyString" } |
There was a problem hiding this comment.
| name { "MyString" } | |
| name { Faker::Lorem.word } |
Use faker instead of hard coded value
| @@ -0,0 +1,8 @@ | |||
| # frozen_string_literal: true | |||
There was a problem hiding this comment.
As the model has been removed and moved to enum, this file may be removed
| factory :question do | ||
| heading { "MyString" } | ||
| statement { "MyText" } | ||
| category { nil } | ||
| difficulty_level { nil } | ||
| test_data { "" } | ||
| circuit_boilerplate { "" } | ||
| end |
There was a problem hiding this comment.
Use Faker . Ref- https://github.com/faker-ruby/faker?tab=readme-ov-file#usage
For test_data and circuit_boilerplate, we may put a blank JSON instead of blank string
| @@ -0,0 +1 @@ | |||
| export OBJC_DISABLE_INITIALIZE_FORK_SAFETY=YES | |||
tanmoysrt
changed the title
Fixes #4966
Added scehams , routes and controller for question management system and tested