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

Fix indenting for translated loops and ors #673

Merged
merged 2 commits into from
Apr 20, 2016
Merged

Conversation

alexpop
Copy link
Contributor

@alexpop alexpop commented Apr 20, 2016

Translated tests like these:

control "uno" do
  shadow.users(/.*/).entries.each do |entry|
    describe entry do
    its(:passwords) { should match /.+/ }
  end
  end
end
control "due" do
  describe.one do
    describe xinetd_conf.services("chargen").socket_types("dgram") do
    it { should be disabled  }
  end
    describe package("xinetd") do
    it { should_not be_installed  }
  end
end

become after this code change:

control "uno" do
  shadow.users(/.*/).entries.each do |entry|
    describe entry do
      its(:passwords) { should match /.+/ }
    end
  end
end
control "due" do
  describe.one do
    describe xinetd_conf.services("chargen").socket_types("dgram") do
      it { should be disabled  }
    end
    describe package("xinetd") do
      it { should_not be_installed  }
    end
  end
end

@alexpop alexpop added the Type: Enhancement Improves an existing feature label Apr 20, 2016
@arlimus
Copy link
Contributor

arlimus commented Apr 20, 2016

Kudos @alexpop for fixing this 👍

@arlimus arlimus merged commit 9000fcd into master Apr 20, 2016
@arlimus arlimus deleted the ap/fix-loop-indent branch April 20, 2016 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Improves an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants