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

ANW-1919 port agents specs to capybara #3168

Merged

Conversation

thimios
Copy link
Collaborator

@thimios thimios commented Mar 20, 2024

Description

Related JIRA Ticket or GitHub Issue

https://archivesspace.atlassian.net/browse/ANW-1919

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have read the CONTRIBUTING document.
  • I have authority to submit this code.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@thimios thimios force-pushed the ANW-1919-frontend-spec-selenium-spec-agents_spec.rb branch 3 times, most recently from ecc1150 to c815503 Compare April 4, 2024 14:42
@thimios thimios force-pushed the ANW-1919-frontend-spec-selenium-spec-agents_spec.rb branch 3 times, most recently from 1889f80 to 4e08004 Compare April 16, 2024 13:46
@thimios thimios force-pushed the ANW-1919-frontend-spec-selenium-spec-agents_spec.rb branch from 4e08004 to f2c0c35 Compare April 19, 2024 16:55
@thimios thimios marked this pull request as ready for review April 19, 2024 17:39
Copy link
Collaborator

@brianzelip brianzelip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thimios Nice work! There are a few points of follow up...

frontend/spec/features/agents_spec.rb Outdated Show resolved Hide resolved
end

it 'hides agent_other_agency_codes from form' do
expect(page).to_not have_css '#agent_person_agent_sources'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The selector here should read '#agent_person_agent_other_agency_codes'.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is actually a duplicate example that I will remove. But you are right there that it needs to check that no '#agent_person_agent_other_agency_codes' are there, which is actually already done three examples above. @brianzelip

frontend/spec/features/agents_spec.rb Show resolved Hide resolved
@thimios thimios force-pushed the ANW-1919-frontend-spec-selenium-spec-agents_spec.rb branch from f2c0c35 to 8a17bd3 Compare April 29, 2024 09:44
@thimios thimios requested a review from brianzelip April 29, 2024 10:27
Copy link
Collaborator

@brianzelip brianzelip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@brianzelip brianzelip merged commit 9fea6f9 into master Apr 29, 2024
20 checks passed
@brianzelip brianzelip deleted the ANW-1919-frontend-spec-selenium-spec-agents_spec.rb branch April 29, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants