Presubmit: run tests within test step, not benchmark (#253)

Not sure how this happened, but I think this is better.

Author: ulfjack

.github/workflows/presubmit.yml

@@ -37,6 +37,11 @@ jobs:
     - name: Test
       run: |
         bazel test //ryu/... //src/...
+        bazel test --copt=-DRYU_OPTIMIZE_SIZE //ryu/...
+        bazel test --copt=-DRYU_ONLY_64_BIT_OPS //ryu/...
+        bazel test --copt=-DRYU_OPTIMIZE_SIZE --copt=-DRYU_ONLY_64_BIT_OPS //ryu/...
+        bazel test --copt=-DRYU_FLOAT_FULL_TABLE //ryu/...
+        bazel test --copt=-DRYU_ONLY_64_BIT_OPS --copt=-DRYU_32_BIT_PLATFORM //ryu/...
 
     - name: Leak checker (macOS)
       if: matrix.os == 'macos-latest' && success()
@@ -48,17 +53,13 @@ jobs:
     # machines are configured and what other things are run on the same
     # instances.
     - name: Benchmark
+      shell: bash
       run: |
         bazel run -c opt //ryu/benchmark:ryu_benchmark --
         bazel run -c opt //ryu/benchmark:ryu_printf_benchmark -- -samples=200
-        bazel test --copt=-DRYU_ONLY_64_BIT_OPS //ryu/...
         bazel run -c opt --copt=-DRYU_ONLY_64_BIT_OPS //ryu/benchmark:ryu_benchmark --
         bazel run -c opt --copt=-DRYU_ONLY_64_BIT_OPS //ryu/benchmark:ryu_printf_benchmark -- -samples=200
-        bazel test --copt=-DRYU_OPTIMIZE_SIZE //ryu/...
         bazel run -c opt --copt=-DRYU_OPTIMIZE_SIZE //ryu/benchmark:ryu_benchmark --
         bazel run -c opt --copt=-DRYU_OPTIMIZE_SIZE //ryu/benchmark:ryu_printf_benchmark -- -samples=200
-        bazel test --copt=-DRYU_OPTIMIZE_SIZE --copt=-DRYU_ONLY_64_BIT_OPS //ryu/...
         bazel run -c opt --copt=-DRYU_OPTIMIZE_SIZE --copt=-DRYU_ONLY_64_BIT_OPS //ryu/benchmark:ryu_benchmark
         bazel run -c opt --copt=-DRYU_OPTIMIZE_SIZE --copt=-DRYU_ONLY_64_BIT_OPS //ryu/benchmark:ryu_printf_benchmark -- -samples=200
-        bazel test -c opt --copt=-DRYU_FLOAT_FULL_TABLE //ryu/...
-        bazel test -c opt --copt=-DRYU_ONLY_64_BIT_OPS --copt=-DRYU_32_BIT_PLATFORM //ryu/...