nushell: restore original test

[waldyrious]

As discussed in #46505, the test for this formula had to be changed due to the behavior of Ctrl+D being mapped to Ctrl+C. The behavior of Ctrl+D has been restored in Nushell v0.6.0, so the test can now be restored.

• Have you followed the guidelines for contributing?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

[waldyrious]

The test is still failing but I can't understand what the issue is. Any insight, @samford, @chenrui333?

Here's the output that I get:

❯ brew test nushell
Testing nushell
==> /usr/local/Cellar/nushell/0.6.0/bin/nu
Error: nushell: failed
An exception occurred within a child process:
  Test::Unit::AssertionFailedError: <"/private/tmp/nushell-test-20191127-74955-h9wnlq> 2\n" +
"/private/tmp/nushell-test-20191127-74955-h9wnlq> CTRL-D\n"> expected but was
<"/private/tmp/nushell-test-20191127-74955-h9wnlq> 2\n" +
"/private/tmp/nushell-test-20191127-74955-h9wnlq> ">.

diff:
  /private/tmp/nushell-test-20191127-74955-h9wnlq> 2
- /private/tmp/nushell-test-20191127-74955-h9wnlq> CTRL-D
?                                                 ^^^^^^^
+ /private/tmp/nushell-test-20191127-74955-h9wnlq>
?                                                 ^
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/gems/2.6.0/gems/test-unit-3.2.9/lib/test/unit/assertions.rb:55:in `block in assert_block'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/gems/2.6.0/gems/test-unit-3.2.9/lib/test/unit/assertions.rb:1636:in `_wrap_assertion'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/gems/2.6.0/gems/test-unit-3.2.9/lib/test/unit/assertions.rb:53:in `assert_block'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/gems/2.6.0/gems/test-unit-3.2.9/lib/test/unit/assertions.rb:240:in `assert_equal'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/nushell.rb:23:in `block in <class:Nushell>'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1740:in `block (3 levels) in run_test'
/usr/local/Homebrew/Library/Homebrew/utils.rb:478:in `with_env'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1739:in `block (2 levels) in run_test'
/usr/local/Homebrew/Library/Homebrew/formula.rb:865:in `with_logging'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1738:in `block in run_test'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1977:in `block in mktemp'
/usr/local/Homebrew/Library/Homebrew/mktemp.rb:57:in `block in run'
/usr/local/Homebrew/Library/Homebrew/mktemp.rb:57:in `chdir'
/usr/local/Homebrew/Library/Homebrew/mktemp.rb:57:in `run'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1976:in `mktemp'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1732:in `run_test'
/usr/local/Homebrew/Library/Homebrew/test.rb:30:in `block in <main>'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/2.6.0/timeout.rb:93:in `block in timeout'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/2.6.0/timeout.rb:33:in `block in catch'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/2.6.0/timeout.rb:33:in `catch'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/2.6.0/timeout.rb:33:in `catch'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/lib/ruby/2.6.0/timeout.rb:108:in `timeout'
/usr/local/Homebrew/Library/Homebrew/test.rb:29:in `<main>'

[Bo98]

As mentioned in #46505 (comment), I think it's just a case that "CTRL-D" doesn't get printed out anymore, though the functionality still works. So it's just a case of removing the "CTRL-D" text from the output assertion as described in the comment.

[samford]

Please remove CTRL-D\n from the end of the assertion text here (keeping the space before it).

[samford]

Yes, I tested this locally with 0.6.0 and the previous test simply needs to be updated to remove CTRL-D\n from the end of the assertion text to pass.

[waldyrious]

Thanks guys! I made the change locally and indeed the test now pass. I've updated the PR accordingly and will now mark it as ready for review/merge.