Find me in app/views/recipients/index.html.erb
diff --git a/config/application.rb b/config/application.rb index 66d7296..674fb06 100644 --- a/config/application.rb +++ b/config/application.rb @@ -1,7 +1,6 @@ require File.expand_path('../boot', __FILE__) require "rails" -# Pick the frameworks you want: require "active_model/railtie" require "active_job/railtie" require "active_record/railtie" @@ -9,25 +8,12 @@ require "action_mailer/railtie" require "action_view/railtie" require "sprockets/railtie" -# require "rails/test_unit/railtie" -# Require the gems listed in Gemfile, including any gems -# you've limited to :test, :development, or :production. Bundler.require(*Rails.groups) module Mailboxer class Application < Rails::Application - # Settings in config/environments/* take precedence over those specified here. - # Application configuration should go into files in config/initializers - # -- all .rb files in that directory are automatically loaded. - # Set Time.zone default to the specified zone and make Active Record auto-convert to this zone. - # Run "rake -D time" for a list of tasks for finding time zone names. Default is UTC. - # config.time_zone = 'Central Time (US & Canada)' - - # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded. - # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s] - # config.i18n.default_locale = :de config.action_mailer.default_url_options = { host: 'ca-mailboxer.herokuapp.com' } @@ -36,6 +22,9 @@ class Application < Rails::Application generate.assets false generate.view_specs false generate.test_framework :rspec + generate.helper_specs false + generate.routing_specs false + generate.controller_specs false end end end diff --git a/config/routes.rb b/config/routes.rb index 1abf899..939541b 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -1,5 +1,6 @@ Rails.application.routes.draw do + get 'recipients/index' devise_for :users get 'welcome/index' diff --git a/coverage/.last_run.json b/coverage/.last_run.json new file mode 100644 index 0000000..6589a0e --- /dev/null +++ b/coverage/.last_run.json @@ -0,0 +1,5 @@ +{ + "result": { + "covered_percent": 33.71 + } +} diff --git a/coverage/.resultset.json b/coverage/.resultset.json new file mode 100644 index 0000000..4ec2c6f --- /dev/null +++ b/coverage/.resultset.json @@ -0,0 +1,179 @@ +{ + "RSpec": { + "coverage": { + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/models/user.rb": [ + 1, + null, + null, + 1, + null, + null, + 1, + 1, + 1, + null, + 1, + null, + 1, + 0, + null, + null, + 1, + 0, + null, + null, + null, + null, + null, + null, + null, + null, + null, + null, + null, + null + ], + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/models/application_record.rb": [ + 1, + 1, + null + ], + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/helpers/application_helper.rb": [ + 1, + 1, + 1, + 1, + 0, + 0, + 0, + null, + null, + 1, + 0, + null, + null + ], + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/helpers/mailbox_helper.rb": [ + 1, + 1, + null, + null, + 0, + null, + null + ], + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/controllers/welcome_controller.rb": [ + 1, + 1, + null, + null + ], + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/controllers/application_controller.rb": [ + 1, + 1, + null, + 1, + null, + 1, + null, + 1, + null, + 1, + 0, + null, + null, + 1, + 0, + null, + null, + 1, + null, + 1, + 4, + 4, + null, + null + ], + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/controllers/mailbox_controller.rb": [ + 0, + 0, + null, + 0, + 0, + 0, + 0, + null, + 0, + 0, + 0, + 0, + null, + 0, + 0, + 0, + 0, + 0 + ], +<<<<<<< HEAD + "/Users/simaboreiri/boot_camp/legacy_code_june_2020/app/controllers/conversations_controller.rb": [ +======= + "/Users/Viktor/CraftAcademy/legacy_code_june_2020/app/controllers/conversations_controller.rb": [ +>>>>>>> 55c2f82e8aa26a4e525d709eaa792f81a930a5fc + 0, + 0, + null, + 0, + 0, + null, + 0, + 0, + 0, + 0, + 0, + 0, + null, + 0, + 0, +<<<<<<< HEAD +======= + null, +>>>>>>> 55c2f82e8aa26a4e525d709eaa792f81a930a5fc + 0, + 0, + null, + 0, + 0, + 0, + 0, + 0, + null, + 0, + 0, + 0, + 0, + null, + 0, + 0, + 0, + 0, + null, + 0, + null, + 0, + 0, + 0, + null, + 0, + 0, + 0, + null, + 0 + ] + }, +<<<<<<< HEAD + "timestamp": 1593090312 +======= + "timestamp": 1593086556 +>>>>>>> 55c2f82e8aa26a4e525d709eaa792f81a930a5fc + } +} diff --git a/coverage/.resultset.json.lock b/coverage/.resultset.json.lock new file mode 100644 index 0000000..e69de29 diff --git a/db/migrate/20200625153035_create_recipients.rb b/db/migrate/20200625153035_create_recipients.rb new file mode 100644 index 0000000..a7a3116 --- /dev/null +++ b/db/migrate/20200625153035_create_recipients.rb @@ -0,0 +1,9 @@ +class CreateRecipients < ActiveRecord::Migration[5.2] + def change + create_table :recipients do |t| + t.string :name + + t.timestamps + end + end +end diff --git a/db/schema.rb b/db/schema.rb index 22fce58..4195b59 100644 --- a/db/schema.rb +++ b/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: 20160322134147) do +ActiveRecord::Schema.define(version: 2020_06_25_153035) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -65,6 +65,12 @@ t.index ["receiver_id", "receiver_type"], name: "index_mailboxer_receipts_on_receiver_id_and_receiver_type" end + create_table "recipients", force: :cascade do |t| + t.string "name" + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + end + create_table "users", id: :serial, force: :cascade do |t| t.string "email", default: "", null: false t.string "encrypted_password", default: "", null: false diff --git a/lib/tasks/ci.rake b/lib/tasks/ci.rake new file mode 100644 index 0000000..4e716d6 --- /dev/null +++ b/lib/tasks/ci.rake @@ -0,0 +1,9 @@ +unless Rails.env.production? + require 'rspec/core/rake_task' + require 'coveralls/rake/task' + Coveralls::RakeTask.new + namespace :ci do + desc 'Run all tests and generate a merged coverage report' + task tests: [:spec, 'coveralls:push'] + end +end \ No newline at end of file diff --git a/password b/password new file mode 100644 index 0000000..e69de29 diff --git a/spec/factories/recipients.rb b/spec/factories/recipients.rb new file mode 100644 index 0000000..8608a7f --- /dev/null +++ b/spec/factories/recipients.rb @@ -0,0 +1,5 @@ +FactoryBot.define do + factory :recipient do + name { 'Name'} + end +end \ No newline at end of file diff --git a/spec/factories/user.rb b/spec/factories/user.rb new file mode 100644 index 0000000..7592e7e --- /dev/null +++ b/spec/factories/user.rb @@ -0,0 +1,7 @@ +FactoryBot.define do + factory :user do + email {'user@mail.com'} + password { 'password'} + name { 'Name'} + end +end \ No newline at end of file diff --git a/spec/feature/sending_message_spec.rb b/spec/feature/sending_message_spec.rb new file mode 100644 index 0000000..da10d25 --- /dev/null +++ b/spec/feature/sending_message_spec.rb @@ -0,0 +1,24 @@ +require "rails_helper" + +feature 'User can send a message' do + context 'Compose a message' + before do + create(:user, email: 'user@mail.com', password: '12345678', name: "Simon") + visit root_path + click_on 'Login' + fill_in "Email", :with => "user@mail.com" + fill_in "Password", :with => "12345678" + click_on "Log in" + click_on "Inbox" + click_on "Compose" + #create(:user, name: 'Simon') + select "Simon", :from => "Recipients" + fill_in "Subject", :with => "Test" + fill_in "Message", :with => "This is a test message." + click_on "Send Message" + end + + it 'user can send a message' do + expect(page).to have_content 'Your message was successfully sent!' + end +end \ No newline at end of file diff --git a/spec/feature/user_can_log_in_feature_spec.rb b/spec/feature/user_can_log_in_feature_spec.rb new file mode 100644 index 0000000..d424f76 --- /dev/null +++ b/spec/feature/user_can_log_in_feature_spec.rb @@ -0,0 +1,28 @@ +require "rails_helper" + +feature 'User can log in' do + context 'Sign in to account' + before do + create(:user, email: 'user@mail.com', password: '1234') + visit user_session_path +end + +it 'displays a log in button' do + expect(page).to have_content 'Login' + click_on 'Login' +end + +it 'displays a email form' do + expect(page).to have_content 'Email' +end + +it 'fills in email and password' do + fill_in "Email", :with => "admin@admin.com" + fill_in "Password", :with => "password" + click_on "Log in" +end + + +end + + diff --git a/spec/feature/user_can_log_out_spec.rb b/spec/feature/user_can_log_out_spec.rb new file mode 100644 index 0000000..e7d4fc0 --- /dev/null +++ b/spec/feature/user_can_log_out_spec.rb @@ -0,0 +1,16 @@ +require "rails_helper" + +feature 'User can log out' do + context 'Sign out of account' do + let(:user) { create(:user, name: 'Peter', password: '23456789', email: 'abc@mail.com') } + before do + login_as(user, scope: :user) + visit root_path + click_on 'Logout' + end + + it 'Displays a successful signout message' do + expect(page).to have_content 'Signed out successfully.' + end +end +end \ No newline at end of file diff --git a/spec/models/recipient_spec.rb b/spec/models/recipient_spec.rb new file mode 100644 index 0000000..3fc1d1a --- /dev/null +++ b/spec/models/recipient_spec.rb @@ -0,0 +1,16 @@ + +RSpec.describe Recipient, type: :model do + describe 'db table' do + it {is_expected.to have_db_column :name} + end + + describe 'validations' do + it {is_expected.to validate_presence_of :name} + end + + describe 'Factory' do + it 'should have valid factory' do + expect(FactoryBot.create(:recipient)).to be_valid + end + end +end diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb new file mode 100644 index 0000000..7468523 --- /dev/null +++ b/spec/models/user_spec.rb @@ -0,0 +1,20 @@ +require 'rails_helper' + +RSpec.describe User, type: :model do + describe 'db table' do + it {is_expected.to have_db_column :email} + it {is_expected.to have_db_column :encrypted_password} + it {is_expected.to have_db_column :name} + end + + describe 'validations' do + it {is_expected.to validate_presence_of :email} + it {is_expected.to validate_presence_of :password} + end + + describe 'Factory' do + it 'should have valid factory' do + expect(FactoryBot.create(:user)).to be_valid + end + end +end \ No newline at end of file diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb new file mode 100644 index 0000000..a3980bf --- /dev/null +++ b/spec/rails_helper.rb @@ -0,0 +1,27 @@ +require 'spec_helper' +ENV['RAILS_ENV'] ||= 'test' +require File.expand_path('../config/environment', __dir__) + +abort("The Rails environment is running in production mode!") if Rails.env.production? +require 'rspec/rails' +begin + ActiveRecord::Migration.maintain_test_schema! +rescue ActiveRecord::PendingMigrationError => e + puts e.to_s.strip + exit 1 +end +RSpec.configure do |config| + config.fixture_path = "#{::Rails.root}/spec/fixtures" + config.use_transactional_fixtures = true + config.infer_spec_type_from_file_location! + config.filter_rails_from_backtrace! + config.include FactoryBot::Syntax::Methods + config.include Warden::Test::Helpers + + Shoulda::Matchers.configure do |config| + config.integrate do |with| + with.test_framework :rspec + with.library :rails + end +end +end diff --git a/spec/requests/recipients_request_spec.rb b/spec/requests/recipients_request_spec.rb new file mode 100644 index 0000000..4d3f27b --- /dev/null +++ b/spec/requests/recipients_request_spec.rb @@ -0,0 +1,12 @@ +require 'rails_helper' + +RSpec.describe "Recipients", type: :request do + + describe "GET /index" do + it "returns http success" do + get "/recipients/index" + expect(response).to have_http_status(:success) + end + end + +end diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb new file mode 100644 index 0000000..3cd93ed --- /dev/null +++ b/spec/spec_helper.rb @@ -0,0 +1,16 @@ + +RSpec.configure do |config| + + config.expect_with :rspec do |expectations| + + expectations.include_chain_clauses_in_custom_matcher_descriptions = true + end + + config.mock_with :rspec do |mocks| + + mocks.verify_partial_doubles = true + end + + config.shared_context_metadata_behavior = :apply_to_host_groups + +end diff --git a/spec/user_can_log_in_spec.rb b/spec/user_can_log_in_spec.rb new file mode 100644 index 0000000..649fb62 --- /dev/null +++ b/spec/user_can_log_in_spec.rb @@ -0,0 +1,15 @@ +require "rails_helper" + +feature 'User can log in' do + context 'Sign in to account' + before do + visit root_path +end + +it 'displays a log in button' do + expect(page).to have_content 'Login' + click_on 'Login' +end +end + +