From f2393771a62c8df6ade9691202fecc47f301b1e5 Mon Sep 17 00:00:00 2001 From: Fabian Affolter Date: Mon, 23 Aug 2021 09:30:12 +0200 Subject: [PATCH] python3Packages.aws-sam-translator: enable tests --- .../aws-sam-translator/default.nix | 42 ++++++++++++++----- 1 file changed, 31 insertions(+), 11 deletions(-) diff --git a/pkgs/development/python-modules/aws-sam-translator/default.nix b/pkgs/development/python-modules/aws-sam-translator/default.nix index 9da3fe3a160b..8bf823e83f0a 100644 --- a/pkgs/development/python-modules/aws-sam-translator/default.nix +++ b/pkgs/development/python-modules/aws-sam-translator/default.nix @@ -1,10 +1,14 @@ { lib -, buildPythonPackage -, fetchPypi -, pythonOlder , boto3 +, buildPythonPackage , enum34 +, fetchFromGitHub , jsonschema +, mock +, parameterized +, pytestCheckHook +, pythonOlder +, pyyaml , six }: @@ -12,23 +16,39 @@ buildPythonPackage rec { pname = "aws-sam-translator"; version = "1.38.0"; - src = fetchPypi { - inherit pname version; - sha256 = "sha256-Dsrdqc9asjGPV/ElMYGiFR5MU8010hcXqSPAdaWmXLY="; + src = fetchFromGitHub { + owner = "aws"; + repo = "serverless-application-model"; + rev = "v${version}"; + sha256 = "0nn9jfqz13kzmxm0r9vy24p8sqxv3mrm5d3lx7ah6rc581q8nv1k"; }; - # Tests are not included in the PyPI package - doCheck = false; - propagatedBuildInputs = [ boto3 jsonschema six - ] ++ lib.optionals (pythonOlder "3.4") [ enum34 ]; + ] ++ lib.optionals (pythonOlder "3.4") [ + enum34 + ]; + + postPatch = '' + substituteInPlace pytest.ini \ + --replace " --cov samtranslator --cov-report term-missing --cov-fail-under 95" "" + ''; + + checkInputs = [ + mock + parameterized + pytestCheckHook + pyyaml + ]; + + pythonImportsCheck = [ "samtranslator" ]; meta = with lib; { - homepage = "https://github.com/awslabs/serverless-application-model"; description = "Python library to transform SAM templates into AWS CloudFormation templates"; + homepage = "https://github.com/awslabs/serverless-application-model"; license = licenses.asl20; + maintainers = with maintainers; [ ]; }; }