Closes AgileVentures/MetPlus_tracker#532 #488
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
patmbolger
changed the title
patmbolger
pushed a commit
that referenced
this pull request
Jan 27, 2017
* changed selenium to poltergeist tests * converted tests to poltergeist * added waits * more waits * added minimum expected for select find() * reduce wait time * increased poltergeist timeout param * changed comment to prefer poltergeist over selenium * added explicit labels for fields * added back selenium for 2 scenarios * backed out changes to jobs form * changed tabs to spaces * Closes AgileVentures/MetPlus_tracker#412 (#461) * interim progress * added task creation * added policy test * wip - adding tests * more rspec tests * added cuke test, renamed feature file * removed cuke step * rubocop fixes * corrected action * rubocop fixes * added comment (#464) * Closes AgileVentures/MetPlus_tracker#509 (#465) * Add missing tests for OTHER JD APPLY * Change to 2 spaces * Closes AgileVentures/MetPlus_tracker#500 (#468) Refactor create and update in jobs_controller * Closes AgileVentures/MetPlus_tracker#487 refactor download resume (#456) * Add step file * pre-fetch commit * For Pull Request * re-commit * Active Jobs * AR Not Found Exception * removed unused instance variable * Closes AgileVentures/MetPlus_tracker#508 (#466) * parameter for agency admin email * remove rspec example file * corrected from-address * add counter_cache to job_skills * add test to skill_spec * update houndci violations * made all emails valid for cuke tests * added wait for 2 tests * removed 100 year old birth date from test * increased capybara wait time * jobseeker notifications * changes to event.rb * add notification to jobseeker * fixing errors * rubocop implement * remove extra spaces * added new method app for job * created new jobseeker mailer revoke * added event * add method jobseekermailer * Created new jonseeker preview mailer * Remove spaces * rubo implement * Resolve rubo issues * updates for spec * Resolve issues rspec test * migrate seed and updates event * removed Factory creation * Resolved conflicts * changes for jobseeker templateview * fix rspec test * Resolved mergeconflicts * Add action for job developer to match job seekers against a job * Add job policy and policy specs * Refactor code and fix rubocop violations * Handle nil job_seeker_ids Fixes the issue when a user bypasses the javascript validation and force submits the form without selecting a job seeker * Add controller tests * Fixed rspec test * removed 100 year old birth date from test * increased capybara wait time * Remove job category placeholder from job_info view closes issue #498 * Resolve bug caused when agency person updates his/her email address Fix minor rubocop warnings in controller and controller spec * Add cuke tests * Flash message when updating email address (#477) * added crucher stub files (#475) * added crucher stub files * updates and rebase code * fixed errors * UI Assets (#481) * 506 company person profile update (#476) * Add step file * pre-fetch commit * For Pull Request * re-commit * Active Jobs * AR Not Found Exception * removed unused instance variable * 506 PR * remover byebug from email validator * fixed job_seeker_spec dob validation * Changed Flash message for Company person email update * company people rspec * Closes AgileVentures/MetPlus_tracker#522 (#479) * Change from Update Agency Person/Update Company Person to Update Profile * fix failing cucumber tests * Closes AgileVentures/MetPlus_tracker#477 (#459) * Closes AgileVentures/MetPlus_tracker#477 * update reviews from PR * handle nil jobseeker address attributes * make address_is_empty method take a parameter Makes it clear we are checking job_seeker_params for presence of address attributes * test address attributes for job_seeker while editing * resolve merge conflict * make tests pass * WIP: speed up cucumber tests (#487) * added code from website one * db clean strategy, refactor cuke scenarios * changes to speed up cuke tests * Closes AgileVentures/MetPlus_tracker#533 (#488) * added wait for XHR request completion * increased wait time * Stub external services (#483) * fixed spelling of raise (#489) * Js update 517 (#485) * updates for js * updates for seed and job_seeker feature * fixed links * updates for jsfeature
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.