Unverified Commit fd6c5273 authored by Jeremy Fleischman's avatar Jeremy Fleischman
Browse files

mycli: skip broken test 

While I was in here, I ported this code to use `pytestCheckHook`.

This fixes https://github.com/NixOS/nixpkgs/issues/211415 (maybe this is
more of a workaround?).

This build was failing due to a broken test upstream. I've instructed
nix to skip that test for now.

I also changed the `py.test` to plain `pytest`. I'm not familiar with
the difference, but according to [this SO
answer](https://stackoverflow.com/a/41893170/1739415), it sounds like
`pytest` is the new hotness and should be used wherever possible.
parent 50981b14

pkgs/tools/admin/mycli/default.nix

+13 −5
Original line number Diff line number Diff line
@@ -30,16 +30,24 @@ buildPythonApplication rec { 
    sqlparse

  ];



  nativeCheckInputs = [ pytest glibcLocales ];

  nativeCheckInputs = [ pytestCheckHook glibcLocales ];



  checkPhase = ''

  preCheck = ''

    export HOME=.

    export LC_ALL="en_US.UTF-8"



    py.test \

      --ignore=mycli/packages/paramiko_stub/__init__.py

  '';



  disabledTestPaths = [

    "mycli/packages/paramiko_stub/__init__.py"

  ];



  disabledTests = [

    # Note: test_auto_escaped_col_names is currently failing due to a bug upstream.

    # TODO: re-enable this test once there is a fix upstream. See

    # https://github.com/dbcli/mycli/issues/1103 for details.

    "test_auto_escaped_col_names"

  ];



  postPatch = ''

    substituteInPlace setup.py \

      --replace "cryptography == 36.0.2" "cryptography"