Journalist can assign an article to a category

app/controllers/articles_controller.rb

@@ -10,6 +10,7 @@ def new
 
   def create
     @article = Article.new(article_params)
+    add_categories_to_article
     if @article.save
       flash[:success] = "Article was successfully created."
       redirect_to @article
@@ -47,6 +48,21 @@ def destroy
 
   private
 
+  def find_articles_and_categories
+      @article = Article.find(params[:id])
+      @categories = Category.all
+  end
+
+  def add_categories_to_article
+    categories = []
+    categories << params[:article][:categories]
+
+    categories.each do |category_id|
+      category = Category.find_by(id: category_id)
+      @article.categories.include?(category) ? next : @article.categories << category
+    end
+  end
+
   def article_params
     params[:article].permit(:title, :content)
   end

app/models/article.rb

@@ -1,4 +1,5 @@
 class Article < ApplicationRecord
   validates :title, :content, presence: true
   has_many :comments, dependent: :destroy
+  has_and_belongs_to_many :categories
 end

app/models/category.rb

@@ -0,0 +1,3 @@
+class Category < ApplicationRecord
+    has_and_belongs_to_many :artilces
+end

app/views/articles/new.html.haml

@@ -9,3 +9,7 @@
     = form.text_area :content, id: :article_content
   %p
     = form.submit value: "Create Article"
+  %p
+    = form.label :categories
+    %br/
+    = form.select(:categories, Category.all.collect {|category| [category.name, category.id] }, {}, {multipe: true, id: 'categories'})

app/views/articles/show.html.haml

@@ -16,6 +16,8 @@
   = form.text_area :content, cols: 50, rows: 5, id: 'comment_content'
   %p= form.submit 'Submit comment'
 
+- @article.categories.each do |category|
+  %p= category.name
 %p
   = link_to 'Edit Article', edit_article_path(@article)
   = link_to 'Delete Article', article_path(@article), method: :delete

db/migrate/20180322195300_create_categories.rb

@@ -0,0 +1,9 @@
+class CreateCategories < ActiveRecord::Migration[5.2]
+  def change
+    create_table :categories do |t|
+      t.string :name
+
+      t.timestamps
+    end
+  end
+end

db/migrate/20180322210143_create_join_table_article_category.rb

@@ -0,0 +1,8 @@
+class CreateJoinTableArticleCategory < ActiveRecord::Migration[5.2]
+  def change
+    create_join_table :articles, :categories do |t|
+      t.index [:article_id, :category_id]
+      t.index [:category_id, :article_id]
+    end
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2018_03_18_155404) do
+ActiveRecord::Schema.define(version: 2018_03_22_210143) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -20,6 +20,22 @@
     t.text "content"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.float "latitude"
+    t.float "longitude"
+    t.string "address"
+  end
+
+  create_table "articles_categories", id: false, force: :cascade do |t|
+    t.bigint "article_id", null: false
+    t.bigint "category_id", null: false
+    t.index ["article_id", "category_id"], name: "index_articles_categories_on_article_id_and_category_id"
+    t.index ["category_id", "article_id"], name: "index_articles_categories_on_category_id_and_article_id"
+  end
+
+  create_table "categories", force: :cascade do |t|
+    t.string "name"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
   end
 
   create_table "comments", force: :cascade do |t|
@@ -45,6 +61,9 @@
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.boolean "subscriber", default: false
+    t.float "latitude"
+    t.float "longitude"
+    t.text "address"
     t.index ["email"], name: "index_users_on_email", unique: true
     t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
   end

features/step_definitions/category_steps.rb

@@ -0,0 +1,13 @@
+Given("the following categories exists") do |table|
+  table.hashes.each do |category|
+    create(:category, category)
+  end
+end
+
+Given("I am on the Create Article page") do
+  visit new_article_path
+end
+
+Then("I select {string} from {string}") do |category, select_box|
+  select(category, from: select_box)
+end

features/support/env.rb

@@ -12,15 +12,14 @@
 end
 
 Cucumber::Rails::Database.javascript_strategy = :truncation
-
+World(FactoryBot::Syntax::Methods)
 Chromedriver.set_version '2.36'
 
 
 Capybara.register_driver :selenium do  |app|
   options = Selenium::WebDriver::Chrome::Options.new(
       implicit_wait: 60,
-      args: %w(disable-popup-blocking disable-infobars),
-      binary: '/Applications/Google Chrome 2.app/Contents/MacOS/Google Chrome'
+      args: %w( headless disable-popup-blocking disable-infobars)
   )
 
   Capybara::Selenium::Driver.new(

features/user_can_asign_categories_to_articles.feature

@@ -0,0 +1,29 @@
+Feature: Better orginized websitet with categories
+  As a subscriber
+  In order to be able to find content of interest
+  I would like to be able to filter articles by category
+
+Background:
+  Given following article exists
+    | title               | content                       |
+    | A Whole New World   | A new fantastic point of view |
+  And the following user exists
+    | email           | password    |
+    | [email protected] | OsloOslo123 |
+  And the following categories exists
+    | name    |
+    | Fashion |
+    | Tech    |
+  And I am logged in as "[email protected]"
+
+Scenario: User creates a new article and asign it to a category
+  Given I am on the Create Article page
+  Then I fill in "Title" with "A Whole New Article"
+  And I fill in "Content" with "A new fantastic Article"
+  And I select "Fashion" from "categories"
+  And I select "Tech" from "categories"
+  When I click "Create Article" button
+  Then I should be on "A Whole New Article" page
+  And I should see "A Whole New Article"
+  And I should see "A new fantastic Article"
+  And I should see "Fashion"

spec/factories/categories.rb

@@ -0,0 +1,5 @@
+FactoryBot.define do
+  factory :category do
+    name "MyString"
+  end
+end

spec/models/article_spec.rb

@@ -12,6 +12,10 @@
     end
   end
 
+  describe "Assosiation" do
+    it {is_expected.to have_and_belong_to_many :categories}
+  end
+
   describe 'Validations' do
     it { is_expected.to validate_presence_of :title }
     it { is_expected.to validate_presence_of :content }

spec/models/category_spec.rb

@@ -0,0 +1,18 @@
+require 'rails_helper'
+
+RSpec.describe Category, type: :model do
+  describe "Factory" do
+    it "should have a valid factory" do
+      article = create(:category)
+      expect(create(:category)).to be_valid
+    end
+  end
+
+  describe "Assosiation" do
+    it {is_expected.to have_and_belong_to_many :articles}
+  end
+
+  describe 'DB columns' do
+    it { is_expected.to have_db_column :name }
+  end
+end

spec/models/comment_spec.rb

@@ -13,8 +13,8 @@
 
   describe "Factory" do
     it "should have a valid factory" do
-      article = FactoryBot.create(:article)
-      expect(FactoryBot.create(:comment, article: article)).to be_valid
+      article = create(:article)
+      expect(create(:comment, article: article)).to be_valid
     end
   end
 end