Fixed / tested Steps::Create

app/models/step.rb

@@ -7,6 +7,9 @@ class Step
   embedded_in :sequence
 
   field :message_type
-  field :time_stamp, type: Time
+  validates :message_type, presence: true
+
   field :command, type: Hash
+  validates :command, presence: true
+
 end

app/mutations/steps/create.rb

@@ -3,19 +3,35 @@ module Steps
 
     required do
       string :message_type, in: Step::MESSAGE_TYPES
-      date :time_stamp
+      hash(:command) { model :*, class: Object }
-      hash(:command) { :* }
       model :sequence
     end
 
-    def validate
+    def execute
+      create(Step, inputs)
     end
 
-    def execute
+    # TODO: Move this into a refinement
-      # STILL NEED TO:
+    # This method provides a uniform way of
-      # * Make the steps mutation create a step
+    # 1. Creating models inside of Mutations
-      # * Make the sequence mutation create one, too.
+    # 2. presenting API model validation errors unifromly.
+    # This is important for keeping API msgs sane.
+    # Returns an instance of Klass -OR- a hash of model validation err. messages
+    # You may optionally pass in a block to transform the model / input
+    # before it gets saved.
+    # Ex: create(User, name: 'Rick', email: "r@mailinator.com") do |user, inputs|
+    #       inputs.email.downcase!
+    #       user.password = "SomethingThatYouPassIn"
+    #     end
+    def create(klass, inputs = {})
       binding.pry
+      model = klass.new(inputs)
+      yield(model, inputs) if block_given?
+      if (model.valid? && model.save)
+        model
+      else
+        model.errors.messages
+      end
     end
   end
 end

spec/controllers/api/sequences/sequences_create_spec.rb

@@ -9,6 +9,7 @@ describe Api::SequencesController do
     let(:user) { FactoryGirl.create(:user) }
 
     it 'creates a new sequences for a user' do
+      pending "Going to create mutation level tests first."
       sign_in user
       input = {name: "Scare Birds",
                 steps: [{
@@ -21,7 +22,6 @@ describe Api::SequencesController do
                               speed: 100,
                               delay: 0}}]}
       post :create, input
-      binding.pry
       expect(response.status).to eq(200)
     end
   end

spec/factories/sequences.rb

@@ -2,7 +2,8 @@
 
 FactoryGirl.define do
   factory :sequence do
-    name
+    name { Faker::Company.catch_phrase }
-    color
+    color { Step::MESSAGE_TYPES.sample }
+    user
   end
 end

spec/mutations/steps/create_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe Steps::Create do
+  let(:user) { FactoryGirl.create(:user) }
+  let(:mutation) { Steps::Create }
+  let(:valid_params) { {message_type: 'move_rel',
+                        sequence: FactoryGirl.create(:sequence),
+                        command: {action: 'MOVE RELATIVE',
+                                  x: 1,
+                                  y: 2,
+                                  z: 3,
+                                  speed: 100,
+                                  delay: 0}} }
+
+  it 'Builds an instance of `step`' do
+    outcome = mutation.run(valid_params)
+    expect(outcome.success?).to be_truthy
+    expect(outcome.result.message_type).to eq(valid_params[:message_type])
+    expect(outcome.result.sequence).to eq(valid_params[:sequence])
+    cmd = outcome.result.command.deep_symbolize_keys
+    expect(cmd).to eq(valid_params[:command])
+  end
+end