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

not to_be_true == true #1

Open
jonhiggs opened this issue Sep 5, 2015 · 2 comments
Open

not to_be_true == true #1

jonhiggs opened this issue Sep 5, 2015 · 2 comments

Comments

@jonhiggs
Copy link

jonhiggs commented Sep 5, 2015

I would expect each of these assertions to pass:

expect $(false) to_be_false
expect $(false) not to_be_true
expect $(true) to_be_true
expect $(true) not to_be_false

however expect $(false) not to_be_true fails.

@holmesjr
Copy link
Owner

holmesjr commented Sep 6, 2015

Putting $(false) in the expectation executes the false command too early. Currently, to_be_true doesn't report on the exit status after it runs, it executes the statement as part of the check.

Try expect not_to_be_true false.

Also, there currently isn't a to_be_false. If not to_be_true isn't enough it could be added.

@holmesjr
Copy link
Owner

holmesjr commented Sep 6, 2015

The main use would be something like this:

expect to_be_true stub_called aws

i.e. its original intention was as a helper for stub.sh

holmesjr added a commit that referenced this issue Mar 30, 2023
* Create ci.yml

* remove pwd from path in makefile

* fix makefile

* fix mskefile

* working

* correct 'workflow' badge

* test beta also

* workflow test beta branch also

---------

Co-authored-by: James Holmes <[email protected]>
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

No branches or pull requests

2 participants