Unit test and runtime behaviour of controller differ

Can you post the controller code? The link labeled “Controller” above is actually a schema module.

I have zero evidence to support this, but my suspicion is that the controller doesn’t agree with the test about what key to use in the params.

Sorry, that was the wrong paste

So if I understand your comment correctly, it’s not using the correct primary key? I’m just very confused about why these three test cases aren’t working as expected, but they are the same as the other model/controller I have set up.

thanks for the help so far though

  def create(conn, job_control_params) do

Try dumping out what you’re getting in job_control_params here when you run the tests. Does it match what JobControls.create_job_control/1 expects?

I don’t believe it will: create_job_control is called from fixture(:job_control) with parameters shaped like

%{comment: ..., control: ..., value: ...}

but in the tests this code:

    test "renders job_control when data is valid", %{conn: conn} do
      conn = post(conn, Routes.job_control_path(conn, :create), job_control: @create_attrs)

will pass (unless use FarmqWeb, :controller is doing something nonstandard) parameters shaped like:

%{"job_control" => %{"comment" => ..., "control" => ..., "value" => ...}}

cast will ignore parameters it doesn’t understand, so it will filter out ALL the input here.

A typical idiom in controller actions looks like:

  def create(conn, %{"job_control" => job_control_params}) do
    with {:ok, %JobControl{} = job_control} <- JobControls.create_job_control(job_control_params) do

...


  def update(conn, %{"id" => id, "job_control" => job_control_params}) do
    job_control = JobControls.get_job_control!(id)

    with {:ok, %JobControl{} = job_control} <- JobControls.update_job_control(job_control, job_control_params) do

I forgot to mention that I updated the original comment with the controller code. The controller code is the same for this controller as well as the other one. I also think that I didn’t edit the create function either.

So I think that the update function is as expected. I printed the contents of job_control_params and it’s what I’d expect it to be

I’d be less confused if my other controller/tests didn’t pass and if using the API via a client wasn’t working as expected

Found the issue. You were absolutely correct in your assumption about the controller being odd. For whatever reason, in the erroring controller I had

def update(conn, job_control_params) do
instead of
def update(conn, %{"job_control" => job_control_params}) do

Same for the create function. Not sure why it was like this but I’ve fixed it now and the tests pass.

Thanks for the help!