From d9c6007dbdf4b3292d912ddb3a4bb16543e7bcb9 Mon Sep 17 00:00:00 2001
From: Bradley Dice <bdice@bradleydice.com>
Date: Wed, 18 Aug 2021 17:09:53 -0500
Subject: [PATCH] Remove JobsCursor.next. (#604)

---
 signac/contrib/project.py | 19 -------------------
 tests/test_project.py     | 18 ------------------
 2 files changed, 37 deletions(-)

diff --git a/signac/contrib/project.py b/signac/contrib/project.py
index be145eeb6..737169394 100644
--- a/signac/contrib/project.py
+++ b/signac/contrib/project.py
@@ -1950,25 +1950,6 @@ def __iter__(self):
             self._project, self._project._find_job_ids(self._filter)
         )
 
-    def next(self):
-        """Return the next element.
-
-        This function is deprecated. Users should use ``next(iter(..))`` instead.
-        .. deprecated:: 0.9.6
-
-        """
-        warnings.warn(
-            "Calling next() directly on a JobsCursor is deprecated! Use next(iter(..)) instead.",
-            DeprecationWarning,
-        )
-        if self._next_iter is None:
-            self._next_iter = iter(self)
-        try:
-            return next(self._next_iter)
-        except StopIteration:
-            self._next_iter = None
-            raise
-
     def groupby(self, key=None, default=None):
         """Group jobs according to one or more state point parameters.
 
diff --git a/tests/test_project.py b/tests/test_project.py
index 8224ef7f2..6289c9110 100644
--- a/tests/test_project.py
+++ b/tests/test_project.py
@@ -325,24 +325,6 @@ def test_find_jobs_JobsCursor_contains(self):
         for sp in statepoints:
             assert self.project.open_job(sp) in cursor_doc
 
-    @pytest.mark.filterwarnings(
-        r"ignore:Calling next\(\) directly on a JobsCursor is deprecated!"
-    )
-    def test_find_jobs_next(self):
-        statepoints = [{"a": i} for i in range(5)]
-        for sp in statepoints:
-            self.project.open_job(sp).init()
-        jobs = self.project.find_jobs()
-        for i in range(2):  # run this twice
-            jobs_ = set()
-            for i in range(len(self.project)):
-                job = jobs.next()
-                assert job in self.project
-                jobs_.add(job)
-            with pytest.raises(StopIteration):
-                job = jobs.next()
-            assert jobs_ == set(self.project)
-
     def test_find_jobs_arithmetic_operators(self):
         for i in range(10):
             self.project.open_job(dict(a=i)).init()