Skip to content

Commit 2a1f30e

Browse files
committed
fix: change news item and test docstring.
1 parent e9c39fe commit 2a1f30e

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

news/deprecate-assignUniqueLabels.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
**Added:**
22

3-
* No News Added: deprecate assignUniqueLabels method
3+
* Added `diffpy.structure.Structure.assign_unique_labels` in replace of `assignUniqueLabels`
44

55
**Changed:**
66

77
* <news item>
88

99
**Deprecated:**
1010

11-
* <news item>
11+
* Deprecated `diffpy.structure.Structure.assignUniqueLabels` for removal in 4.0.0
1212

1313
**Removed:**
1414

tests/test_structure.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -121,18 +121,15 @@ def test___copy__(self):
121121
# return
122122

123123
def test_assignUniqueLabels(self):
124-
"""Duplicate test with the deprecated assignUniqueLabels method.
125-
126-
Remove this test in version 4.0.0
127-
"""
124+
"""Check Structure.assignUniqueLabels()"""
128125
self.assertEqual("", "".join([a.label for a in self.stru]))
129126
self.stru.assignUniqueLabels()
130127
self.assertEqual("C1", self.stru[0].label)
131128
self.assertEqual("C2", self.stru[1].label)
132129
return
133130

134131
def test_assign_unique_labels(self):
135-
"""Check Structure.assignUniqueLabels()"""
132+
"""Check Structure.assign_unique_labels()"""
136133
self.assertEqual("", "".join([a.label for a in self.stru]))
137134
self.stru.assign_unique_labels()
138135
self.assertEqual("C1", self.stru[0].label)

0 commit comments

Comments
 (0)