Remove --worker and --build as mandatory arguments

Considering that we, now, can re-run the same builds using `--reproduce`
it makes sense to simplify the re-run command we report in sentry.

bin/rspecq

@@ -2,6 +2,8 @@
 require "optparse"
 require "rspecq"
 
+DEFAULT_WORKER = "foo".freeze
+DEFAULT_BUILD = "bar".freeze
 DEFAULT_REDIS_HOST = "127.0.0.1".freeze
 DEFAULT_REPORT_TIMEOUT = 3600 # 1 hour
 DEFAULT_MAX_REQUEUES = 3
@@ -127,8 +129,8 @@ OptionParser.new do |o|
   end
 end.parse!
 
-opts[:build] ||= ENV["RSPECQ_BUILD"]
-opts[:worker] ||= ENV["RSPECQ_WORKER"]
+opts[:build] ||= ENV["RSPECQ_BUILD"] || DEFAULT_BUILD
+opts[:worker] ||= ENV["RSPECQ_WORKER"] || DEFAULT_WORKER
 opts[:seed] ||= ENV["RSPECQ_SEED"]
 opts[:redis_host] ||= ENV["RSPECQ_REDIS"] || DEFAULT_REDIS_HOST
 opts[:timings] ||= env_set?("RSPECQ_UPDATE_TIMINGS")

lib/rspecq/queue.rb

@@ -173,8 +173,8 @@ def job_rerun_command(job)
       jobs = redis.lrange(key("queue", "jobs_per_worker", worker), 0, -1)
       seed = redis.hget(key("worker_seed"), worker)
 
-      "DISABLE_SPRING=1 DISABLE_BOOTSNAP=1 bin/rspecq --build 1 " \
-        "--worker foo --seed #{seed} --max-requeues 0 --fail-fast 1 " \
+      "DISABLE_SPRING=1 DISABLE_BOOTSNAP=1 bin/rspecq " \
+        "--seed #{seed} --max-requeues 0 --fail-fast 1 " \
         "--reproduction #{jobs.join(' ')}"
     end
 

test/test_reporter.rb

@@ -32,8 +32,8 @@ def test_flakey_suite
 
     assert_match "Flaky jobs detected", output
     assert_match "./spec/foo_spec.rb:2 @ #{worker_id}", output
-    assert_match "DISABLE_SPRING=1 DISABLE_BOOTSNAP=1 bin/rspecq --build 1 " \
-        "--worker foo --seed 1234 --max-requeues 0 --fail-fast 1 --reproduction " \
+    assert_match "DISABLE_SPRING=1 DISABLE_BOOTSNAP=1 bin/rspecq " \
+        "--seed 1234 --max-requeues 0 --fail-fast 1 --reproduction " \
         "./spec/foo_spec.rb ./spec/foo_spec.rb[1:1] ./spec/foo_spec.rb[1:1]", output
     refute_match "Failed examples", output
   end