Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fake data script #313

Closed
wants to merge 210 commits into from
Closed

Fake data script #313

wants to merge 210 commits into from

Conversation

razztech
Copy link
Contributor

What this PR does:

This pull request implements two scripts that populate the database for testing / development. One of them creates fake schools and the other one creates fake teachers. The PR also includes rspec tests that test the functionality/reliability of the script.

Include screenshots, videos, etc.

Here are some screenshots with the fake data generated by the scripts to populate the database.
Screenshot 2024-04-24 at 4 55 58 PM
Screenshot 2024-04-24 at 4 56 16 PM

Who authored this PR?

@razztech

How should this PR be tested?

rails db:populate_teachers
bundle exec rspec

  • What do the specs/features test?
  • Overall script functionality: mainly testing that new entries in the db are created for teachers and schools.

Are there any complications to deploying this?

Run bundle install since the PR adds the Faker gem used for the fake data.

New 💎: Faker

razztech and others added 30 commits March 7, 2024 19:41
@razztech
Added frontend + db logic for country field in international schools
f2ee4d2
@razztech
Added some new frontend logic
1000250
@razztech
Made changes to UI logic and updated tests
277e4de
@razztech
Update tests to use country dropdown
43b0a3d
@perryzjc
Merge pull request #39 from cs169/187129573-change-teacher-status
bfe7a19 
187129573 change teacher status
@razztech
All tests are passing
f3cd0cd
@razztech
Finished table country display
dc852f3
@razztech
Added filtering with a different column
fa48319
@razztech
Fixed rubocop
fadef9a
@razztech
Added state textfield test for international schools
1320a4e
@razztech
Merge branch 'main' into 187216522-international-schools
2fe9505
@razztech
Fixed merge issues for cucumber tests
b2c8e47
@razztech
Added international school test, moved custom web step to form_steps.rb:
c9df771
@razztech
Fixed rubocop
5d8654b
@razztech
Added country validation
6660f2e
@razztech
Added new test for country
39123b8
@razztech
Changed UI based on Michael's feedback
d27c514
@razztech
Made UI changes requested by Michael 2
29de296
@razztech
Changed from VanillaJS to jQuerry suggested by Arush
2c0363d
@ArushC
Merge pull request #38 from cs169/187186207/iter1-golden-repo-fixes
5135d22 
187186207/iter1 golden repo fixes
@perryzjc
Further change from VanillaJS to jQuerry suggested by Arush
677bb6e
@perryzjc
Add cucumber test for unexpected user behavior
772f2df
@ArushC
fix glitch
05d989e
@ArushC
resolve merge conflict
a2ea667
@ArushC
ok actually fixed merge conflict now
fa1155b
@ArushC
no choice, have to resolve conflict manually
a155d65
@ArushC
Merge branch 'main' into 187216522-international-schools
da5f122
@ArushC
Merge pull request #42 from cs169/187216522-international-schools
71b92b6 
Country field added for international schools
@ArushC
added languages dropdown without selectize
4af15e7
@ArushC
added selectize thingie
db87716
ArushC and others added 29 commits April 16, 2024 18:57
@ArushC
Merge branch '187384622/new-email-model-pr3' of github.com:cs169/BJC-…
dab1b54 
…Teacher-Tracker-App into teacher-merge-feature
@ArushC
Merge pull request #52 from cs169/187384622/new-email-model-pr3
565307d 
187384622/new email model - PR3 - Add schema migration to drop old email columns and Adjust Relevant Dependencies
@ArushC
FINALLY cucumber tests plz work plz plz
f23eca4
@ArushC
I need this to work
7fb905a
@perryzjc
Update backend
b4b2f75
@perryzjc
Update frontend
82f4c4c
@perryzjc
Add cucumber tests
5cc4676
@perryzjc
Fix rubocop
7f31e97
@perryzjc
Add rspec for email validation
9b318f9
@perryzjc
Merge branch 'main' into 187384622/new-email-model-pr4
5e098f4
@ArushC
Merge pull request #54 from cs169/187384622/new-email-model-pr4
096f456 
187384622/new email model - PR4 - Now add multi-personal-email feature
@perryzjc
Merge branch '187384622/new-email-model' into pd-frontend+backend
d6bb607
@perryzjc
Merge branch '187384622/new-email-model-pr2' into pd-frontend+backend
c909c95
@perryzjc
Merge branch '187384622/new-email-model-pr3' into pd-frontend+backend
45bc536
@perryzjc
Fix cucumber tests to adapt new schema from new email address model
b0fa3d5
@perryzjc
Remove unused variable
436485f
@perryzjc
Add more rspec tests
417adea
@perryzjc
Merge pull request #48 from cs169/pd-frontend+backend
a1fda8e 
Professional Development Feature
@razztech
Wrote script to generate random fake teachers
c6d08c4
@razztech
Finished part of the fake data script
8584fa2
@razztech
Added script to generate schools
1ba6d0b
@ArushC
fixed merge conflicts wrt main
46a8388
@ArushC
fixed confirm form resubmission error
15b71fb
@ArushC
made buttons smaller on merge page
749e254
@ArushC
make personal email plural
867b4bd
@ArushC
Merge pull request #51 from cs169/teacher-merge-feature
ca6c172 
Teacher merge feature
@razztech
Split the populate teacher task into smaller files
99c3dd5
@razztech
Added rspec test for each creation script
332ee31
@razztech
Merge branch 'main' into fake-data-script
3dbfc14
@razztech razztech closed this Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@razztech @perryzjc @ArushC @JacksonXu33 @cycomachead