Feature ad making

app/controllers/ad_controller.rb

@@ -18,7 +18,7 @@ def create
       flash[:notice] = 'Ad registered!'
       redirect_to(ad_index_path)
     else
-      render('ad/new')
+      render(new_ad_path)
     end
   end
 
@@ -28,7 +28,7 @@ def update
       flash[:notice] = 'Ad updated!'
       redirect_to(ad_index_path)
     else
-      render('ad/edit')
+      render(edit_ad_path)
     end
   end
 

app/views/ad/edit.html.erb

@@ -1,13 +1,13 @@
 <%= render  partial: "errorlog" %>
 <%= form_for(@ad ,url: ad_path(@ad),html: {method: "patch",class: "form-group"}) do |f| %> <%#＃入稿フォーム %>
   <div class="form-group">
-    <p>text</p>
+    <p>広告文</p>
     <%= f.text_area :text,value: @ad.text,class: "form-control" %>
-    <p>price</p>
+    <p>単価</p>
     <%= f.number_field :price,value: @ad.price %>
-    <p>advertiser_id</p>
+    <p>広告主ID</p>
     <%= f.number_field :advertiser_id,value: @ad.advertiser_id %>
-    <p>image</p>
+    <p>イメージ</p>
     <%= f.file_field :image,value: @ad.image,class: "form-control-file" %>
     <%= f.submit class: "btn btn-primary",value: "更新" %>
   </div>

app/views/ad/index.html.erb

@@ -2,13 +2,13 @@
   <div class="each_ad"> <%# 広告を一つ一つ表示 %>
     <% @ads.each do |ad| %>
       <li>
-        <span>ad_id :<%= ad.id %></span>
+        <span>広告ID:<%= ad.id %></span>
       </li>
       <li>
-        <span>text :<%= ad.text %></span>
+        <span>広告文 :<%= ad.text %></span>
       </li>
       <li>
-        <span>price :<%= ad.price %></span>
+        <span>単価 :<%= ad.price %></span>
       </li>
       <li>
         <%= image_tag ad.image.to_s %>

app/views/ad/new.html.erb

@@ -1,13 +1,13 @@
 <%= render  partial: "errorlog" %>
 <%= form_for(@ad ,url: ad_index_path,html: {method: "post",class: "form-group"}) do |f| %> <%#＃入稿フォーム %>
   <div class="form-group">
-    <p>text</p>
+    <p>広告文</p>
     <%= f.text_area :text,value: @ad.text,class: "form-control" %>
-    <p>price</p>
+    <p>単価</p>
     <%= f.number_field :price,value: @ad.price %>
-    <p>advertiser_id</p>
+    <p>広告主ID</p>
     <%= f.number_field :advertiser_id,value: @ad.advertiser_id %>
-    <p>image</p>
+    <p>イメージ</p>
     <%= f.file_field :image,value: @ad.image,class: "form-control-file" %>
     <%= f.submit class: "btn btn-primary",value: "入稿" %>
   </div>

db/schema.rb

@@ -1,4 +1,3 @@
-# frozen_string_literal: true
 # This file is auto-generated from the current state of the database. Instead
 # of editing this file, please use the migrations feature of Active Record to
 # incrementally modify your database, and then regenerate this schema definition.
@@ -11,16 +10,15 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2019_06_04_073026) do
-  create_table 'ads', force: :cascade do |t|
-    t.integer 'advertiser_id', default: 0, null: false
-    t.string 'image', default: '', null: false
-    t.integer 'price', default: 0, null: false
-    t.string 'text', default: '', null: false
-    t.datetime 'created_at', null: false
-    t.datetime 'updated_at', null: false
-    t.integer 'click'
-    t.integer 'imp'
-    t.integer 'cv'
+ActiveRecord::Schema.define(version: 2019_05_28_085221) do
+
+  create_table "ads", force: :cascade do |t|
+    t.integer "advertiser_id", default: 0, null: false
+    t.string "image", default: "", null: false
+    t.integer "price", default: 0, null: false
+    t.string "text", default: "", null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
   end
+
 end

spec/controllers/ad_controller_spec.rb

@@ -58,8 +58,7 @@
           'price' => nil,
           'advertiser_id' => nil,
           'image' => nil
-        },
-                                 id: @ad.id }
+        },id: @ad.id }
         expect(response).to render_template :edit
       end
 
@@ -89,8 +88,7 @@
           'price' => 1010101,
           'advertiser_id' => 1010101,
           'image' => Rack::Test::UploadedFile.new(File.join(Rails.root, 'spec/fixture/image.jpg'))
-        },
-                                 id: @ad.id }
+        },id: @ad.id }
         expect(response).to redirect_to(ad_index_path)
       end
     end
@@ -122,8 +120,7 @@
           'price' => nil,
           'advertiser_id' => nil,
           'image' => nil
-        },
-                                 id: @ad.id }
+        },id: @ad.id }
         expect(response).to render_template :edit
       end
     end

spec/features/path_spec.rb

@@ -50,7 +50,7 @@
 describe 'Buttons' do
   context 'after new ad created' do
     before do
-      visit 'ad/new'
+      visit new_ad_path
       fill_in 'ad_text', with: 'TextForm'
       fill_in 'ad_price', with: '1000'
       fill_in 'ad_advertiser_id', with: '1'