diff options
| author | Teddy Wing | 2017-11-15 16:22:05 +0100 | 
|---|---|---|
| committer | Teddy Wing | 2017-11-15 16:22:05 +0100 | 
| commit | 35def4911d4eb7f6f3c2aadb6b1983993f9293f6 (patch) | |
| tree | 78f663c91004a5add71d984a2621fdc472fcb944 /spec/db | |
| parent | 28b4f328e455a425c6222cdd249515ea958a45d3 (diff) | |
| download | chouette-core-35def4911d4eb7f6f3c2aadb6b1983993f9293f6.tar.bz2 | |
schema_spec: Remove unnecessary capture
Thanks to Robert for suggesting `(?:...)`. I didn't know that syntax,
happy I learned something new. Here, I had used parentheses for the "or"
(`|`), but didn't need the captured value. To do the "or" without
capturing, you can use the `?:` inside the start of the parentheses.
Refs #4951
Diffstat (limited to 'spec/db')
| -rw-r--r-- | spec/db/schema_spec.rb | 2 | 
1 files changed, 1 insertions, 1 deletions
| diff --git a/spec/db/schema_spec.rb b/spec/db/schema_spec.rb index b2f259abd..74e92bebe 100644 --- a/spec/db/schema_spec.rb +++ b/spec/db/schema_spec.rb @@ -16,7 +16,7 @@ RSpec::Matchers.define :use_bigint_keys do          # Primary key          if line =~ /create_table\s/ && -          !(line =~ /id:\s+(:bigserial|false)/) +          !(line =~ /id:\s+(?::bigserial|false)/)            expected_line = line.sub(/(create_table\s"\w+",\s)/, '\1id: :bigserial, ')          end | 
