diff --git a/copi.owasp.org/lib/copi/cornucopia/vote.ex b/copi.owasp.org/lib/copi/cornucopia/vote.ex index 9cd451776..cb72deb18 100644 --- a/copi.owasp.org/lib/copi/cornucopia/vote.ex +++ b/copi.owasp.org/lib/copi/cornucopia/vote.ex @@ -12,7 +12,7 @@ defmodule Copi.Cornucopia.Vote do @doc false def changeset(vote, attrs) do vote - |> cast(attrs, []) - |> validate_required([]) + |> cast(attrs, [:dealt_card_id, :player_id]) + |> validate_required([:dealt_card_id, :player_id]) end end diff --git a/copi.owasp.org/test/copi/cornucopia/vote_test.exs b/copi.owasp.org/test/copi/cornucopia/vote_test.exs index 043626c4b..2be200ac3 100644 --- a/copi.owasp.org/test/copi/cornucopia/vote_test.exs +++ b/copi.owasp.org/test/copi/cornucopia/vote_test.exs @@ -1,12 +1,22 @@ defmodule Copi.Cornucopia.VoteTest do - use ExUnit.Case, async: true + use Copi.DataCase alias Copi.Cornucopia.Vote - describe "changeset/2" do - test "valid with empty attrs" do - changeset = Vote.changeset(%Vote{}, %{}) + describe "votes" do + @valid_attrs %{dealt_card_id: 1, player_id: Ecto.ULID.generate()} + @invalid_attrs %{dealt_card_id: nil, player_id: nil} + + test "changeset with valid attributes" do + changeset = Vote.changeset(%Vote{}, @valid_attrs) assert changeset.valid? end + + test "changeset with invalid attributes" do + changeset = Vote.changeset(%Vote{}, @invalid_attrs) + refute changeset.valid? + assert "can't be blank" in errors_on(changeset).dealt_card_id + assert "can't be blank" in errors_on(changeset).player_id + end end -end +end \ No newline at end of file