Improve validations

app/models/deployment.rb

 class Deployment < ActiveRecord::Base
   include InternalId
 
-  belongs_to :project
-  belongs_to :environment
+  belongs_to :project, required: true
+  belongs_to :environment, required: true
   belongs_to :user
   belongs_to :deployable, polymorphic: true
 
   validates :sha, presence: true
   validates :ref, presence: true
-  validates :project, associated: true
-  validates :environment, associated: true
 
   delegate :name, to: :environment, prefix: true
 

app/models/environment.rb

 class Environment < ActiveRecord::Base
-  belongs_to :project
+  belongs_to :project, required: true
 
   has_many :deployments
 
@@ -10,8 +10,6 @@ class Environment < ActiveRecord::Base
             format: { with: Gitlab::Regex.environment_name_regex,
                       message: Gitlab::Regex.environment_name_regex_message }
 
-  validates :project, associated: true
-
   def last_deployment
     deployments.last
   end

db/migrate/20160610204157_add_deployments.rb

@@ -6,12 +6,12 @@ class AddDeployments < ActiveRecord::Migration
 
   def change
     create_table :deployments, force: true do |t|
-      t.integer  :iid
-      t.integer  :project_id
-      t.integer  :environment_id
-      t.string   :ref
-      t.boolean  :tag
-      t.string   :sha
+      t.integer  :iid,            null: false
+      t.integer  :project_id,     null: false
+      t.integer  :environment_id, null: false
+      t.string   :ref,            null: false
+      t.boolean  :tag,            null: false
+      t.string   :sha,            null: false
       t.integer  :user_id
       t.integer  :deployable_id
       t.string   :deployable_type

db/migrate/20160610204158_add_environments.rb

@@ -6,7 +6,7 @@ class AddEnvironments < ActiveRecord::Migration
 
   def change
     create_table :environments, force: true do |t|
-      t.integer  :project_id
+      t.integer  :project_id, null: false
       t.string   :name,       null: false
       t.datetime :created_at
       t.datetime :updated_at

db/schema.rb

@@ -384,12 +384,12 @@ ActiveRecord::Schema.define(version: 20160610301627) do
   add_index "deploy_keys_projects", ["project_id"], name: "index_deploy_keys_projects_on_project_id", using: :btree
 
   create_table "deployments", force: :cascade do |t|
-    t.integer  "iid"
-    t.integer  "project_id"
-    t.integer  "environment_id"
-    t.string   "ref"
-    t.boolean  "tag"
-    t.string   "sha"
+    t.integer  "iid",             null: false
+    t.integer  "project_id",      null: false
+    t.integer  "environment_id",  null: false
+    t.string   "ref",             null: false
+    t.boolean  "tag",             null: false
+    t.string   "sha",             null: false
     t.integer  "user_id"
     t.integer  "deployable_id"
     t.string   "deployable_type"
@@ -413,7 +413,7 @@ ActiveRecord::Schema.define(version: 20160610301627) do
   add_index "emails", ["user_id"], name: "index_emails_on_user_id", using: :btree
 
   create_table "environments", force: :cascade do |t|
-    t.integer  "project_id"
+    t.integer  "project_id", null: false
     t.string   "name",       null: false
     t.datetime "created_at"
     t.datetime "updated_at"

spec/factories/deployments.rb

@@ -2,6 +2,7 @@ FactoryGirl.define do
   factory :deployment, class: Deployment do
     sha '97de212e80737a608d939f648d959671fb0a0142'
     ref 'master'
+    tag false
 
     environment factory: :environment