Compare commits
3 commits
0a2f26684b
...
7834dd7188
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
7834dd7188 | ||
|
|
f396604316 | ||
|
|
eb81be50f9 |
11 changed files with 239 additions and 7 deletions
34
app/controllers/day_logs_controller.rb
Normal file
34
app/controllers/day_logs_controller.rb
Normal file
|
|
@ -0,0 +1,34 @@
|
|||
class DayLogsController < ApplicationController
|
||||
def new
|
||||
@day_log = DayLog.new
|
||||
end
|
||||
|
||||
def create
|
||||
@day_log = Current.user.day_logs.build(day_log_params)
|
||||
|
||||
if @day_log.save
|
||||
handle_mood(@day_log.day, params[:day_log][:mode_id])
|
||||
redirect_to root_path
|
||||
else
|
||||
render :new, status: :unprocessable_entity
|
||||
end
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def handle_mood(day, mode_id)
|
||||
return if mode_id.blank?
|
||||
|
||||
mood = Current.user.moods.find_by(recorded_at: day.beginning_of_day..day.end_of_day)
|
||||
|
||||
if mood
|
||||
mood.update(mode_id: mode_id)
|
||||
else
|
||||
Current.user.moods.create(mode_id: mode_id, recorded_at: day.to_datetime)
|
||||
end
|
||||
end
|
||||
|
||||
def day_log_params
|
||||
params.expect(day_log: [ :day, :info ])
|
||||
end
|
||||
end
|
||||
|
|
@ -1,4 +1,19 @@
|
|||
class Mood < ApplicationRecord
|
||||
belongs_to :user
|
||||
belongs_to :mode
|
||||
|
||||
validates :recorded_at, presence: true
|
||||
validate :unique_per_day_and_user
|
||||
|
||||
private
|
||||
|
||||
def unique_per_day_and_user
|
||||
return unless recorded_at && user
|
||||
|
||||
existing = user.moods.where.not(id: id).any? do |mood|
|
||||
mood.recorded_at.to_date == recorded_at.to_date
|
||||
end
|
||||
|
||||
errors.add(:recorded_at, :taken) if existing
|
||||
end
|
||||
end
|
||||
|
|
|
|||
|
|
@ -3,6 +3,7 @@ class User < ApplicationRecord
|
|||
has_many :sessions, dependent: :destroy
|
||||
has_many :moods, -> { order "recorded_at" }
|
||||
has_many :modes
|
||||
has_many :day_logs
|
||||
|
||||
normalizes :email_address, with: ->(e) { e.strip.downcase }
|
||||
|
||||
|
|
|
|||
21
app/views/day_logs/new.html.haml
Normal file
21
app/views/day_logs/new.html.haml
Normal file
|
|
@ -0,0 +1,21 @@
|
|||
%h1 Nouveau journal
|
||||
|
||||
= form_with model: @day_log do |f|
|
||||
- @day_log.errors.full_messages.each do |msg|
|
||||
%p= msg
|
||||
|
||||
.field
|
||||
= f.label :day
|
||||
= f.date_field :day
|
||||
|
||||
.field
|
||||
= f.label :info
|
||||
= f.text_area :info
|
||||
|
||||
.field
|
||||
= label_tag :mode_id, "Mode"
|
||||
= select_tag "day_log[mode_id]",
|
||||
options_from_collection_for_select(Current.user.modes, :id, :label),
|
||||
include_blank: true
|
||||
|
||||
= f.submit
|
||||
|
|
@ -21,4 +21,6 @@ Rails.application.routes.draw do
|
|||
patch "/invite/:token", to: "invitations#update", as: :invitation
|
||||
|
||||
get "/moods", to: "moods#index", as: :dashboard
|
||||
|
||||
resources :day_logs, only: [ :new, :create ]
|
||||
end
|
||||
|
|
|
|||
|
|
@ -1,7 +1,6 @@
|
|||
FactoryBot.define do
|
||||
factory :mode do
|
||||
label { "MyString" }
|
||||
slug { "MyString" }
|
||||
color { "MyString" }
|
||||
user { nil }
|
||||
end
|
||||
|
|
|
|||
|
|
@ -1,7 +1,7 @@
|
|||
FactoryBot.define do
|
||||
factory :mood do
|
||||
mode { "croisiere" }
|
||||
recorded_at { DateTime.now }
|
||||
recorded_at { DateTime.parse("2026-01-15 10:00:00") }
|
||||
association :user
|
||||
association :mode
|
||||
end
|
||||
end
|
||||
|
|
|
|||
|
|
@ -2,6 +2,7 @@ FactoryBot.define do
|
|||
factory :user do
|
||||
sequence(:email_address) { |n| "user#{n}@example.com" }
|
||||
sequence(:username) { |n| "user#{n}" }
|
||||
password_digest { BCrypt::Password.create('password123') }
|
||||
password { "obanonalors" }
|
||||
password_digest { BCrypt::Password.create(password) }
|
||||
end
|
||||
end
|
||||
|
|
|
|||
|
|
@ -1,7 +1,46 @@
|
|||
require 'rails_helper'
|
||||
|
||||
describe Mood, type: :model do
|
||||
it 'works' do
|
||||
expect(true)
|
||||
describe Mood do
|
||||
let(:user) { create(:user) }
|
||||
let(:mode) { create(:mode, user: user) }
|
||||
|
||||
subject { build(:mood, user: user, mode: mode, recorded_at: DateTime.parse("2026-01-15 10:00:00")) }
|
||||
|
||||
it { is_expected.to be_valid }
|
||||
|
||||
describe "validations" do
|
||||
it "is invalid without a user" do
|
||||
subject.user = nil
|
||||
expect(subject).not_to be_valid
|
||||
end
|
||||
|
||||
it "is invalid without a mode" do
|
||||
subject.mode = nil
|
||||
expect(subject).not_to be_valid
|
||||
end
|
||||
|
||||
it "is invalid without a recorded_at" do
|
||||
subject.recorded_at = nil
|
||||
expect(subject).not_to be_valid
|
||||
end
|
||||
end
|
||||
|
||||
describe "uniqueness per day and user" do
|
||||
it "is invalid if a mood already exists for the same day and user" do
|
||||
create(:mood, user: user, mode: mode, recorded_at: DateTime.parse("2026-01-15 18:00:00"))
|
||||
expect(subject).not_to be_valid
|
||||
end
|
||||
|
||||
it "is valid if the same day exists but for a different user" do
|
||||
other_user = create(:user)
|
||||
other_mode = create(:mode, user: other_user)
|
||||
create(:mood, user: other_user, mode: other_mode, recorded_at: DateTime.parse("2026-01-15 10:00:00"))
|
||||
expect(subject).to be_valid
|
||||
end
|
||||
|
||||
it "is valid if the same user has a mood on a different day" do
|
||||
create(:mood, user: user, mode: mode, recorded_at: DateTime.parse("2026-01-14 10:00:00"))
|
||||
expect(subject).to be_valid
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
|||
108
spec/requests/day_logs_spec.rb
Normal file
108
spec/requests/day_logs_spec.rb
Normal file
|
|
@ -0,0 +1,108 @@
|
|||
require 'rails_helper'
|
||||
|
||||
describe "DayLogs", type: :request do
|
||||
let(:user) { create(:user) }
|
||||
let(:mode) { create(:mode, user: user) }
|
||||
let(:day) { Date.parse("2026-01-15") }
|
||||
|
||||
describe "GET /day_logs/new" do
|
||||
context "when not authenticated" do
|
||||
it "redirects to the login page" do
|
||||
get new_day_log_path
|
||||
expect(response).to redirect_to(new_session_path)
|
||||
end
|
||||
end
|
||||
|
||||
context "when authenticated" do
|
||||
before { login_as(user) }
|
||||
|
||||
it "returns http success" do
|
||||
get new_day_log_path
|
||||
expect(response).to have_http_status(:success)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
describe "POST /day_logs" do
|
||||
context "when not authenticated" do
|
||||
it "redirects to the login page" do
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info" } }
|
||||
expect(response).to redirect_to(new_session_path)
|
||||
end
|
||||
end
|
||||
|
||||
context "when authenticated" do
|
||||
before { login_as(user) }
|
||||
|
||||
context "with valid params" do
|
||||
context "without mode" do
|
||||
it "creates a day_log" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info" } }
|
||||
}.to change(DayLog, :count).by(1)
|
||||
end
|
||||
|
||||
it "does not create a mood" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info" } }
|
||||
}.not_to change(Mood, :count)
|
||||
end
|
||||
end
|
||||
|
||||
context "with mode, no existing mood" do
|
||||
it "creates a day_log" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info", mode_id: mode.id } }
|
||||
}.to change(DayLog, :count).by(1)
|
||||
end
|
||||
|
||||
it "creates a mood" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info", mode_id: mode.id } }
|
||||
}.to change(Mood, :count).by(1)
|
||||
end
|
||||
end
|
||||
|
||||
context "with mode, existing mood" do
|
||||
let!(:existing_mood) { create(:mood, user: user, mode: mode, recorded_at: DateTime.parse("2026-01-15 08:00:00")) }
|
||||
let(:other_mode) { create(:mode, user: user) }
|
||||
|
||||
it "creates a day_log" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info", mode_id: other_mode.id } }
|
||||
}.to change(DayLog, :count).by(1)
|
||||
end
|
||||
|
||||
it "does not create a new mood" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info", mode_id: other_mode.id } }
|
||||
}.not_to change(Mood, :count)
|
||||
end
|
||||
|
||||
it "updates the existing mood mode" do
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info", mode_id: other_mode.id } }
|
||||
expect(existing_mood.reload.mode).to eq(other_mode)
|
||||
end
|
||||
end
|
||||
|
||||
it "redirects after creation" do
|
||||
post day_logs_path, params: { day_log: { day: day, info: "Une info" } }
|
||||
expect(response).to have_http_status(:redirect)
|
||||
end
|
||||
end
|
||||
|
||||
context "with invalid params" do
|
||||
it "does not create a day_log" do
|
||||
expect {
|
||||
post day_logs_path, params: { day_log: { day: nil, info: nil } }
|
||||
}.not_to change(DayLog, :count)
|
||||
end
|
||||
|
||||
it "returns unprocessable entity" do
|
||||
post day_logs_path, params: { day_log: { day: nil, info: nil } }
|
||||
expect(response).to have_http_status(:unprocessable_entity)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
12
spec/support/authentication_helper.rb
Normal file
12
spec/support/authentication_helper.rb
Normal file
|
|
@ -0,0 +1,12 @@
|
|||
module AuthenticationHelper
|
||||
def login_as(user)
|
||||
post session_path, params: {
|
||||
email_address: user.email_address,
|
||||
password: user.password
|
||||
}
|
||||
end
|
||||
end
|
||||
|
||||
RSpec.configure do |config|
|
||||
config.include AuthenticationHelper, type: :request
|
||||
end
|
||||
Loading…
Add table
Reference in a new issue