Use RuntimeError instead of ArgumentError if missing Sequel database …

…connection

No argument is being provided, so ArgumentError isn't appropriate.

Make the spec use begin/ensure so that a failure of the new spec
does not make all subsequent specs break.

lib/rodauth/features/base.rb

@@ -264,7 +264,7 @@ def open_account?
     end
 
     def db
-      Sequel::DATABASES.first or raise ArgumentError, "Sequel database connection is missing"
+      Sequel::DATABASES.first or raise "Sequel database connection is missing"
     end
 
     def password_field_autocomplete_value

spec/rodauth_spec.rb

@@ -1232,12 +1232,14 @@ def foo
   end
 
   it "should raise error when sequel database is missing" do
-    database = Sequel::DATABASES.pop
-
-    rodauth {}
-    error = proc { roda {} }.must_raise(ArgumentError)
-    error.message.must_equal "Sequel database connection is missing"
+    begin
+      database = Sequel::DATABASES.pop
 
-    Sequel::DATABASES.push database
+      rodauth {}
+      error = proc { roda {} }.must_raise(RuntimeError)
+      error.message.must_equal "Sequel database connection is missing"
+    ensure
+      Sequel::DATABASES.push database if database
+    end
   end
 end