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.

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

Registration sad path #126

Open
wants to merge 43 commits into
base: main
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
d6af67e
Create pull_request_template.md
LawrenceWhalen Jul 5, 2021
62f0b55
Update pull_request_template.md
LawrenceWhalen Jul 5, 2021
3823fa2
Add refactor option to change type
LawrenceWhalen Jul 5, 2021
709d03f
Add heroku data to travis CI
LawrenceWhalen Jul 5, 2021
b72daab
Start Travis
LawrenceWhalen Jul 6, 2021
85e39b8
Update .travis.yml
LawrenceWhalen Jul 6, 2021
a24541d
Update pull_request_template.md
LawrenceWhalen Jul 6, 2021
4e74e33
Merge pull request #1 from LawrenceWhalen/Whalen-pull-request-template
LawrenceWhalen Jul 6, 2021
d57349f
Create model tests and models
LawrenceWhalen Jul 6, 2021
1b42293
Create and test self reference for user
LawrenceWhalen Jul 6, 2021
5a03b14
Early push for emergency
LawrenceWhalen Jul 6, 2021
8da2503
fix migration of atendee
Aphilosopher30 Jul 6, 2021
8d9ff53
make tests for models relationships
Aphilosopher30 Jul 6, 2021
a1e9724
Merge branch 'main' of github.com:LawrenceWhalen/viewing_party
LawrenceWhalen Jul 7, 2021
df83cf3
Merge branch 'main' into model-test
LawrenceWhalen Jul 7, 2021
fb39cbf
Update gemfile ruby version
LawrenceWhalen Jul 7, 2021
bb766fe
Create and test the user creation process
LawrenceWhalen Jul 7, 2021
6809c32
Merge branch 'main' into creation-setup
LawrenceWhalen Jul 7, 2021
0860123
Merge pull request #22 from LawrenceWhalen/model-test
Aphilosopher30 Jul 7, 2021
dbbef03
Update user_spec.rb
LawrenceWhalen Jul 7, 2021
6a5d5e6
Merge branch 'main' into creation-setup
LawrenceWhalen Jul 7, 2021
bddb74d
Merge branch 'main' of github.com:LawrenceWhalen/viewing_party
LawrenceWhalen Jul 7, 2021
eaff82e
Delete 20210707213534_create_attendee.rb
LawrenceWhalen Jul 7, 2021
0ca8ad5
Merge pull request #23 from LawrenceWhalen/creation-setup
Aphilosopher30 Jul 7, 2021
7ff5771
Merge branch 'main' of github.com:LawrenceWhalen/viewing_party
LawrenceWhalen Jul 7, 2021
c07ddb5
Test and create loggin in and out
LawrenceWhalen Jul 7, 2021
7789ccc
Merge pull request #24 from LawrenceWhalen/login-setup
Aphilosopher30 Jul 8, 2021
24e3531
sad paths for registration
Aphilosopher30 Jul 8, 2021
ca48956
Swap from sessions to cookies and show path to dashboard
LawrenceWhalen Jul 8, 2021
2305e40
upgrade ruby version
Aphilosopher30 Jul 8, 2021
0ccc36d
preperation for murge to main
Aphilosopher30 Jul 8, 2021
b35b3cc
add tests to check that we are adding what we want to the data base
Aphilosopher30 Jul 8, 2021
36ba12a
Setup webmock/vcr and write tests for api
LawrenceWhalen Jul 8, 2021
718af8e
add password confirmation
Aphilosopher30 Jul 8, 2021
120fae6
validate features
Aphilosopher30 Jul 8, 2021
13e8e91
Create test for and pass movie index tests
LawrenceWhalen Jul 8, 2021
4183917
Remove sensative data
LawrenceWhalen Jul 8, 2021
a825363
Merge pull request #26 from LawrenceWhalen/movies-setup
Aphilosopher30 Jul 9, 2021
4152ac4
merge changes into branch
Aphilosopher30 Jul 9, 2021
b823a73
validation tests fix
Aphilosopher30 Jul 9, 2021
d100dcd
Sensitive data
LawrenceWhalen Jul 9, 2021
66fdf56
Sensitive data
LawrenceWhalen Jul 9, 2021
a116193
Sensitive data
LawrenceWhalen Jul 9, 2021
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
Prev Previous commit
Next Next commit
validate features
Aphilosopher30 committed Jul 8, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
commit 120fae6cefc98926e93638856124d5e8c9df60c1
2 changes: 1 addition & 1 deletion app/models/attendee.rb
Original file line number Diff line number Diff line change
@@ -3,7 +3,7 @@ class Attendee < ApplicationRecord
belongs_to :user
belongs_to :movie_party

# validates_presence_of :user, :viewing_party
validates_presence_of :user, :movie_party
# validates :user_id, uniqueness: {scope: :viewing_party_id}

end
2 changes: 1 addition & 1 deletion app/models/friend.rb
Original file line number Diff line number Diff line change
@@ -3,7 +3,7 @@ class Friend < ApplicationRecord
belongs_to :friender, class_name: 'User'
belongs_to :friendee, class_name: 'User'

# validates_presence_of :friender, :friendee
validates_presence_of :friender, :friendee
# validates :friender, uniqueness: {scope: :friendee_id}

end
5 changes: 1 addition & 4 deletions app/models/movie_party.rb
Original file line number Diff line number Diff line change
@@ -4,8 +4,5 @@ class MovieParty < ApplicationRecord
has_many :attendees, dependent: :destroy
has_many :users, through: :attendees

# belongs_to :user

# validates_presence_of :user, :movie_title, :date_time

validates_presence_of :user_id, :movie_title, :date_time
end
5 changes: 5 additions & 0 deletions spec/models/attendee_spec.rb
Original file line number Diff line number Diff line change
@@ -7,4 +7,9 @@
it { should belong_to(:movie_party) }
end

describe 'validations' do
it { should validate_presence_of :user }
it { should validate_presence_of :movie_party }
end

end
5 changes: 5 additions & 0 deletions spec/models/friend_spec.rb
Original file line number Diff line number Diff line change
@@ -7,4 +7,9 @@
it { should belong_to :friender }
end

describe 'validations' do
it { should validate_presence_of :friendee }
it { should validate_presence_of :friender }
end

end
7 changes: 7 additions & 0 deletions spec/models/movie_party_spec.rb
Original file line number Diff line number Diff line change
@@ -7,4 +7,11 @@
it { should have_many(:users) }
end



describe 'validations' do
it { should validate_presence_of :user_id }
it { should validate_presence_of :movie_title }
it { should validate_presence_of :date_time }
end
end
1 change: 0 additions & 1 deletion spec/models/user_spec.rb
Original file line number Diff line number Diff line change
@@ -8,7 +8,6 @@
it { should have_many(:frienders).class_name('User') }

it { should have_many(:attendees).dependent(:destroy) }
# it { should have_many(:movie_parties) }
it { should have_many(:movie_parties).dependent(:destroy)}
end

Empty file removed spec/models/viewing_party_spec.rb
Empty file.