Fixed some things with the test add functions

db/migrations/20230923020649_add_tests_table.php

@@ -14,7 +14,7 @@ final class AddTestsTable extends AbstractMigration {
       ->addTimestamps()
       ->create();
 
-    $table = $this->table('tests_benchmarks');
+    $table = $this->table('test_benchmark');
     $table->addColumn('test_id', 'integer', ['null' => false])
       ->addColumn('benchmark_id', 'integer', ['null' => false])
       ->addForeignKey('test_id', 'tests', 'id', ['delete'=> 'CASCADE', 'update'=> 'CASCADE'])

src/Controllers/TestController.php

@@ -49,13 +49,13 @@ class TestController extends Controller {
     $test->description = $params['test_description'];
     $test->component_id = $params['test_component'];
 
+    $test->save();
+
     // attach benchmarks to test
     foreach ($params['test_benchmarks'] as $b) {
       $test->benchmarks()->attach($b);
     }
 
-    $test->save();
-
     // redirect the user back to the home page
     $routeContext = RouteContext::fromRequest($request);
     $routeParser = $routeContext->getRouteParser();

src/Models/Test.php

@@ -16,7 +16,7 @@ class Test extends Model {
     return $this->hasMany(Result::class);
   }
 
-  public function benchmark() {
+  public function benchmarks() {
     return $this->belongsToMany(Benchmark::class);
   }
 

views/test/add.twig

@@ -37,7 +37,7 @@
         <div class="twelve columns">
           <label>
             Benchmarks:
-            <select id="benchmark-selector" class="u-full-width" name="test_benchmarks" multiple>
+            <select id="benchmark-selector" class="u-full-width" name="test_benchmarks[]" multiple>
               {% for b in benchmarks %}
                 <option value="{{ b.id }}">{{ b.name }}</option>
                 {% endfor %}