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

Updated Specs for Foreman Provider save and Service Dialog Dropdown Options change #1656

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Jul 5, 2017

Updated Specs for Foreman Provider save and Service Dialog Dropdown Options changes.

UI specs need to be updated to match the changes in the foreman and master repos:

Links

ManageIQ/manageiq-providers-foreman#5
ManageIQ/manageiq#15456

@dclarizio dclarizio self-assigned this Jul 5, 2017
@dclarizio dclarizio added the test label Jul 5, 2017
@dclarizio
Copy link

@lgalis looks like one failing test was missed, can you take a look?

@lgalis lgalis force-pushed the update_specs_after_foreman_and_master_updates branch from e6c0643 to 27ae9dd Compare July 5, 2017 22:50
@miq-bot
Copy link
Member

miq-bot commented Jul 5, 2017

Checked commits lgalis/manageiq-ui-classic@b7f52b1~...27ae9dd with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
3 files checked, 0 offenses detected
Everything looks fine. 🏆

@dclarizio dclarizio merged commit 022fa43 into ManageIQ:master Jul 5, 2017
@dclarizio dclarizio added this to the Sprint 64 Ending Jul 10, 2017 milestone Jul 5, 2017
@himdel himdel added the fine/yes label Aug 9, 2017
@himdel
Copy link
Contributor

himdel commented Aug 9, 2017

Added fine/yes since this should fix

  1) ProviderForemanController #save_provider_foreman will not save with a duplicate name
     Failure/Error: expect(assigns(:flash_array).first[:message]).to include("Name has already been taken")
       expected "Configuration_manager.name has already been taken" to include "Name has already been taken"
     # ./spec/controllers/provider_foreman_controller_spec.rb:143:in `block (2 levels) in <top (required)>'
     # ./spec/spec_helper.rb:75:in `block (3 levels) in <top (required)>'
     # ./spec/manageiq/spec/support/evm_spec_helper.rb:34:in `clear_caches'
     # ./spec/spec_helper.rb:75:in `block (2 levels) in <top (required)>'

(gitter ☝️ August 8, 2017 2:48 PM)

simaishi pushed a commit that referenced this pull request Aug 9, 2017
…aster_updates

Updated Specs for Foreman Provider save and Service Dialog Dropdown Options change
(cherry picked from commit 022fa43)
@simaishi
Copy link
Contributor

simaishi commented Aug 9, 2017

Fine backport details:

$ git log -1
commit f0553be7360f37fa7a39635ce476ca4b75363314
Author: Dan Clarizio <[email protected]>
Date:   Wed Jul 5 16:23:28 2017 -0700

    Merge pull request #1656 from lgalis/update_specs_after_foreman_and_master_updates
    
    Updated Specs for Foreman Provider save and Service Dialog Dropdown Options change
    (cherry picked from commit 022fa4315cf28c26baa5c4247e97231d00ad5e8d)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants