Merge pull request #94 from sean-dickinson/add-unique-to-fake

Feat: Add unique option to fake

docsite/source/index.html.md

@@ -141,6 +141,14 @@ Factory.define(:user) do |f|
 end
 ```
 
+##### Unique values with fake
+Passing the unique: true option will use Faker's [unique](https://github.com/faker-ruby/faker#ensuring-unique-values) feature  
+```ruby
+Factory.define(:user) do |f|
+  f.email { fake(:internet, :email, unique: true) }
+end
+```
+
 #### Dependent attributes
 
 Attributes can be based on the values of other attributes:

lib/rom/factory/dsl.rb

@@ -13,17 +13,23 @@ module Factory
     class << self
       # @api private
       def fake(*args, **options)
-        api = fetch_or_store(:faker, *args) do
+        const, method_name = fetch_or_store(:faker, *args) do
           *ns, method_name = args
 
           const = ns.reduce(::Faker) do |obj, name|
             obj.const_get(::Dry::Core::Inflector.camelize(name))
           end
 
-          const.method(method_name)
+          [const, method_name]
         end
 
-        api.(**options)
+        is_unique = options.delete(:unique) { false }
+
+        if is_unique
+          const.unique.public_send(method_name, **options)
+        else
+          const.method(method_name).(**options)
+        end
       end
     end
 
@@ -113,9 +119,13 @@ def timestamps
       #   @example
       #     f.email { fake(:number, :between, from: 10, to: 100) }
       #
+      #   @example
+      #    f.email { fake(:internet, :email, unique: true) }
+      #
       #   @param [Symbol] genre The faker API identifier ie. :internet, :product etc.
       #   @param [Symbol] type The value type to generate
-      #   @param [Hash] options Additional arguments
+      #   @param [Hash] options Additional arguments, including unique: true will generate unique values
+      #
       #
       # @overload fake(genre, subgenre, type, **options)
       #   @example

spec/integration/rom/factory_spec.rb

@@ -919,6 +919,18 @@ class Admin < ROM::Struct
       expect(user.created_at).to_not be(nil)
       expect(user.created_at).to_not be(nil)
     end
+
+    it "uses unique when the unique option is true" do
+      factories.define(:user) do |f|
+        f.first_name { fake(:name) }
+        f.last_name { fake(:name, :last_name) }
+        f.email { fake(:internet, :email, unique: true) }
+        f.timestamps
+      end
+
+      emails = 10.times.map { factories.structs[:user].email }
+      expect(emails.uniq.length).to eq(emails.length)
+    end
   end
 
   context "custom non integer sequence primary_key" do