WIP: Karol's implementation #1

Closed
stan wants to merge 23 commits from karol_master into master
9 changed files with 18 additions and 13 deletions
Showing only changes of commit cd40ef66c6 - Show all commits

View file

@ -3,7 +3,7 @@ class SessionsController < ApplicationController
end
def create
@user = User.find_by(username: params[:username])
@user = User.find_by(email: params[:email])
if @user && @user.authenticate(params[:password])
session[:user_id] = @user.id
end

View file

@ -4,7 +4,7 @@ class UsersController < ApplicationController
end
def create
@user = User.create(params.require(:user).permit(:username, :password))
@user = User.create(params.require(:user).permit(:email, :password))
session[:user_id] = @user.id
redirect_to '/welcome'
end

View file

@ -1,7 +1,7 @@
<h1>Login</h1>
<%= form_tag '/login' do %>
<%= label_tag :username%>
<%= text_field_tag :username %>
<%= label_tag :email%>
<%= text_field_tag :email %>
<%= label_tag :password%>
<%= password_field_tag :password%>
<%= submit_tag "Login"%>

View file

@ -1,6 +1,6 @@
<h1>Welcome</h1>
<% if logged_in? %>
<h1>You are Logged In, <%= current_user.username %></h1>
<h1>You are Logged In, <%= current_user.email %></h1>
<%= button_to "Logout", '/logout', method: :get%>
<%else%>
<%= button_to "Login", '/login', method: :get%>

View file

@ -1,7 +1,7 @@
<h1>Sign Up</h1>
<%= form_for @user do |f|%>
<%= f.label :username%><br>
<%= f.text_field :username%><br>
<%= f.label :email%><br>
<%= f.text_field :email%><br>
<%= f.label :password%><br>
<%= f.password_field :password%><br>
<%= f.submit %>

View file

@ -0,0 +1,5 @@
class ChangeUsernameToEmail < ActiveRecord::Migration[6.1]
def change
rename_column :users, :username, :email
end
end

4
db/schema.rb generated
View file

@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.
ActiveRecord::Schema.define(version: 2021_03_20_130542) do
ActiveRecord::Schema.define(version: 2021_03_20_212922) do
create_table "authors", force: :cascade do |t|
t.string "first_name"
@ -39,7 +39,7 @@ ActiveRecord::Schema.define(version: 2021_03_20_130542) do
end
create_table "users", force: :cascade do |t|
t.string "username"
t.string "email"
t.string "password_digest"
t.datetime "created_at", precision: 6, null: false
t.datetime "updated_at", precision: 6, null: false

View file

@ -3,12 +3,12 @@ require 'rails_helper'
RSpec.describe ApplicationController do
describe 'current_user' do
before(:all) do
User.create(username: 'karol.selak@gmail.com', password: 'abcde')
User.create(email: 'karol.selak@gmail.com', password: 'abcde')
end
context 'when a user is logged in' do
it 'returns the user' do
# TODO
# expect(current_user.username).to eql('karol.selak@gmail.com')
# expect(current_user.email).to eql('karol.selak@gmail.com')
end
end
end

View file

@ -12,11 +12,11 @@ RSpec.describe UsersController do
User.destroy_all
end
subject {
get :create, params: {user: {username: 'karol.selak@gmail.com', password: 'abcde'}}
get :create, params: {user: {email: 'karol.selak@gmail.com', password: 'abcde'}}
}
it 'creates a user' do
subject
expect(User.where(username: 'karol.selak@gmail.com').size).to eql(1)
expect(User.where(email: 'karol.selak@gmail.com').size).to eql(1)
end
it 'redirects to /welcome' do
subject