From 12ef4ebea7c13c668981e526297b3b6ca58ff068 Mon Sep 17 00:00:00 2001
From: Fabian Affolter <mail@fabian-affolter.ch>
Date: Sat, 24 Jun 2023 23:16:03 +0200
Subject: python310Packages.confluent-kafka: add optional-dependencies

- enable tests
- add pythonImportsCheck
- add format
- disable on unsupported Python releases
---
 .../python-modules/confluent-kafka/default.nix     | 66 +++++++++++++++++++---
 1 file changed, 58 insertions(+), 8 deletions(-)

(limited to 'pkgs/development/python-modules/confluent-kafka')

diff --git a/pkgs/development/python-modules/confluent-kafka/default.nix b/pkgs/development/python-modules/confluent-kafka/default.nix
index 0a0433741bd56..03cdb826786ce 100644
--- a/pkgs/development/python-modules/confluent-kafka/default.nix
+++ b/pkgs/development/python-modules/confluent-kafka/default.nix
@@ -1,18 +1,68 @@
-{ lib, buildPythonPackage, fetchPypi, isPy3k, rdkafka, requests, avro3k, avro ? null, futures ? null, enum34 ? null }:
+{ lib
+, avro
+, buildPythonPackage
+, fastavro
+, fetchFromGitHub
+, jsonschema
+, protobuf
+, pyflakes
+, pyrsistent
+, pytestCheckHook
+, pythonOlder
+, rdkafka
+, requests
+, requests-mock
+}:
 
 buildPythonPackage rec {
-  version = "2.1.1";
   pname = "confluent-kafka";
+  version = "2.1.1";
+  format = "setuptools";
+
+  disabled = pythonOlder "3.7";
+
+  src = fetchFromGitHub {
+    owner = "confluentinc";
+    repo = "confluent-kafka-python";
+    rev = "refs/tags/v${version}";
+    hash = "sha256-xnbovQRvbhaYYXnssV0Jy+U9L6BUddIagbup2jdTugY=";
+  };
+
+  buildInputs = [
+    rdkafka
+  ];
 
-  src = fetchPypi {
-    inherit pname version;
-    hash = "sha256-pmXnQDCoXF9CA1OM/nfkARM6hJxLkgUqYCJbJgUqfp0=";
+  propagatedBuildInputs = [
+    requests
+  ];
+
+  passthru.optional-dependencies = {
+    avro = [
+      avro
+      fastavro
+    ];
+    json = [
+      jsonschema
+      pyrsistent
+    ];
+    protobuf = [
+      protobuf
+    ];
   };
 
-  buildInputs = [ rdkafka requests ] ++ (if isPy3k then [ avro3k ] else [ enum34 avro futures ]) ;
+  nativeCheckInputs = [
+    pyflakes
+    pytestCheckHook
+    requests-mock
+  ] ++ lib.flatten (builtins.attrValues passthru.optional-dependencies);
+
+  pythonImportsCheck = [
+    "confluent_kafka"
+  ];
 
-  # No tests in PyPi Tarball
-  doCheck = false;
+  disabledTestPaths = [
+    "tests/integration/"
+  ];
 
   meta = with lib; {
     description = "Confluent's Apache Kafka client for Python";
-- 
cgit 1.4.1