diff options
author | Tim Smith <tsmith@chef.io> | 2019-02-19 10:57:01 -0800 |
---|---|---|
committer | Tim Smith <tsmith@chef.io> | 2019-02-19 10:57:01 -0800 |
commit | 2b596f35c58d9278ab53a5ec781374daeb005b5b (patch) | |
tree | 15b452bbcb191de06037b0cfaa270f28513fd71e | |
parent | c0a342cefb02df7b069f36809a18a0c54d2672da (diff) | |
download | chef-2b596f35c58d9278ab53a5ec781374daeb005b5b.tar.gz |
Fix the comment spec
This was entirely wrong. We're setting the comment to nil and proving that it ends up being the user and not the comment, but then we actually expected it to be the comment. This spec did not due what the description said it did. Now it does.
Signed-off-by: Tim Smith <tsmith@chef.io>
-rw-r--r-- | spec/unit/provider/user/dscl_spec.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/spec/unit/provider/user/dscl_spec.rb b/spec/unit/provider/user/dscl_spec.rb index 4951315da4..b12ea78977 100644 --- a/spec/unit/provider/user/dscl_spec.rb +++ b/spec/unit/provider/user/dscl_spec.rb @@ -560,9 +560,8 @@ ea18e18b720e358e7fbe3cfbeaa561456f6ba008937a30" it "sets the comment field to username" do new_resource.comment nil - expect(provider).to receive(:run_dscl).with("create", "/Users/toor", "RealName", "#mockssuck").and_return(true) + expect(provider).to receive(:run_dscl).with("create", "/Users/toor", "RealName", "toor").and_return(true) provider.dscl_create_comment - expect(new_resource.comment).to eq("#mockssuck") end it "should run run_dscl with create /Users/user PrimaryGroupID to set the users primary group" do |