mirror of
https://github.com/inventree/InvenTree.git
synced 2025-05-15 11:33:08 +00:00
* Squashed commit of the following: commit 52d7ff0f650bbcfa2d93ac96562b44269d3812a7 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 23:03:20 2024 +0100 fixed lookup commit 0d076eaea89dce24f08af247479b3b4dff1b4df3 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 23:03:08 2024 +0100 switched to pathlib for lookup commit 473e75eda205793769946e923748356ffd7e5b4b Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 22:52:30 2024 +0100 fix wrong url response commit fd74f8d703399c19cb3616ea3b2656a50cd7a6e5 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 21:14:38 2024 +0100 switched to ruff for import sorting commit f83fedbbb8de261ff8c706e179519e58e7a91064 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 21:03:14 2024 +0100 switched to single quotes everywhere commit a92442e60e23be0ff5dcf42d222b0d95823ecb9b Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:58:23 2024 +0100 added autofixes commit cc66c93136fcae8a701810a4f4f38ef3b570be61 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:56:47 2024 +0100 enable autoformat commit 1f343606ec1f2a99acf8a37b9900d78a8fb37282 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:42:14 2024 +0100 Squashed commit of the following: commit f5cf7b2e7872fc19633321713965763d1890b495 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:36:57 2024 +0100 fixed reqs commit 9d845bee98befa4e53c2ac3c783bd704369e3ad2 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:32:35 2024 +0100 disable autofix/format commit aff5f271484c3500df7ddde043767c008ce4af21 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:28:50 2024 +0100 adjust checks commit 47271cf1efa848ec8374a0d83b5646d06fffa6e7 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:28:22 2024 +0100 reorder order of operations commit e1bf178b40b3f0d2d59ba92209156c43095959d2 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:01:09 2024 +0100 adapted ruff settings to better fit code base commit ad7d88a6f4f15c9552522131c4e207256fc2bbf6 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 19:59:45 2024 +0100 auto fixed docstring commit a2e54a760e17932dbbc2de0dec23906107f2cda9 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 19:46:35 2024 +0100 fix getattr useage commit cb80c73bc6c0be7f5d2ed3cc9b2ac03fdefd5c41 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 19:25:09 2024 +0100 fix requirements file commit b7780bbd21a32007f3b0ce495b519bf59bb19bf5 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:42:28 2024 +0100 fix removed sections commit 71f1681f55c15f62c16c1d7f30a745adc496db97 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:41:21 2024 +0100 fix djlint syntax commit a0bcf1bccef8a8ffd482f38e2063bc9066e1d759 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:35:28 2024 +0100 remove flake8 from code base commit 22475b31cc06919785be046e007915e43f356793 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:34:56 2024 +0100 remove flake8 from code base commit 0413350f14773ac6161473e0cfb069713c13c691 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:24:39 2024 +0100 moved ruff section commit d90c48a0bf98befdfacbbb093ee56cdb28afb40d Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:24:24 2024 +0100 move djlint config to pyproject commit c5ce55d5119bf2e35e429986f62f875c86178ae1 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:20:39 2024 +0100 added isort again commit 42a41d23afc280d4ee6f0e640148abc6f460f05a Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:19:02 2024 +0100 move config section commit 85692331816348cb1145570340d1f6488a8265cc Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:17:52 2024 +0100 fix codespell error commit 2897c6704d1311a800ce5aa47878d96d6980b377 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 17:29:21 2024 +0100 replaced flake8 with ruff mostly for speed improvements * enable docstring checks * fix docstrings * fixed D417 Missing argument description * Squashed commit of the following: commit d3b795824b5d6d1c0eda67150b45b5cd672b3f6b Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 22:56:17 2024 +0100 fixed source path commit 0bac0c19b88897a19d5c995e4ff50427718b827e Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 22:47:53 2024 +0100 fixed req commit 9f61f01d9cc01f1fb7123102f3658c890469b8ce Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 22:45:18 2024 +0100 added missing toml req commit 91b71ed24a6761b629768d0ad8829fec2819a966 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:49:50 2024 +0100 moved isort config commit 12460b04196b12d0272d40552402476d5492fea5 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:43:22 2024 +0100 remove flake8 section from setup.cfg commit f5cf7b2e7872fc19633321713965763d1890b495 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:36:57 2024 +0100 fixed reqs commit 9d845bee98befa4e53c2ac3c783bd704369e3ad2 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:32:35 2024 +0100 disable autofix/format commit aff5f271484c3500df7ddde043767c008ce4af21 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:28:50 2024 +0100 adjust checks commit 47271cf1efa848ec8374a0d83b5646d06fffa6e7 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:28:22 2024 +0100 reorder order of operations commit e1bf178b40b3f0d2d59ba92209156c43095959d2 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 20:01:09 2024 +0100 adapted ruff settings to better fit code base commit ad7d88a6f4f15c9552522131c4e207256fc2bbf6 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 19:59:45 2024 +0100 auto fixed docstring commit a2e54a760e17932dbbc2de0dec23906107f2cda9 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 19:46:35 2024 +0100 fix getattr useage commit cb80c73bc6c0be7f5d2ed3cc9b2ac03fdefd5c41 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 19:25:09 2024 +0100 fix requirements file commit b7780bbd21a32007f3b0ce495b519bf59bb19bf5 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:42:28 2024 +0100 fix removed sections commit 71f1681f55c15f62c16c1d7f30a745adc496db97 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:41:21 2024 +0100 fix djlint syntax commit a0bcf1bccef8a8ffd482f38e2063bc9066e1d759 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:35:28 2024 +0100 remove flake8 from code base commit 22475b31cc06919785be046e007915e43f356793 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:34:56 2024 +0100 remove flake8 from code base commit 0413350f14773ac6161473e0cfb069713c13c691 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:24:39 2024 +0100 moved ruff section commit d90c48a0bf98befdfacbbb093ee56cdb28afb40d Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:24:24 2024 +0100 move djlint config to pyproject commit c5ce55d5119bf2e35e429986f62f875c86178ae1 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:20:39 2024 +0100 added isort again commit 42a41d23afc280d4ee6f0e640148abc6f460f05a Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:19:02 2024 +0100 move config section commit 85692331816348cb1145570340d1f6488a8265cc Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 18:17:52 2024 +0100 fix codespell error commit 2897c6704d1311a800ce5aa47878d96d6980b377 Author: Matthias Mair <code@mjmair.com> Date: Sun Jan 7 17:29:21 2024 +0100 replaced flake8 with ruff mostly for speed improvements * fix pyproject * make docstrings more uniform * auto-format * fix order * revert url change
304 lines
10 KiB
Python
304 lines
10 KiB
Python
"""Unit tests for the BomItem model."""
|
|
|
|
from decimal import Decimal
|
|
|
|
import django.core.exceptions as django_exceptions
|
|
from django.db import transaction
|
|
from django.test import TestCase
|
|
|
|
import stock.models
|
|
|
|
from .models import BomItem, BomItemSubstitute, Part
|
|
|
|
|
|
class BomItemTest(TestCase):
|
|
"""Class for unit testing BomItem model."""
|
|
|
|
fixtures = [
|
|
'category',
|
|
'part',
|
|
'location',
|
|
'bom',
|
|
'company',
|
|
'supplier_part',
|
|
'part_pricebreaks',
|
|
'price_breaks',
|
|
]
|
|
|
|
def setUp(self):
|
|
"""Create initial data."""
|
|
super().setUp()
|
|
|
|
Part.objects.rebuild()
|
|
|
|
self.bob = Part.objects.get(id=100)
|
|
self.orphan = Part.objects.get(name='Orphan')
|
|
self.r1 = Part.objects.get(name='R_2K2_0805')
|
|
|
|
def test_str(self):
|
|
"""Test the string representation of a BOMItem."""
|
|
b = BomItem.objects.get(id=1)
|
|
self.assertEqual(str(b), '10 x M2x4 LPHS to make BOB | Bob | A2')
|
|
|
|
def test_has_bom(self):
|
|
"""Test the has_bom attribute."""
|
|
self.assertFalse(self.orphan.has_bom)
|
|
self.assertTrue(self.bob.has_bom)
|
|
|
|
self.assertEqual(self.bob.bom_count, 4)
|
|
|
|
def test_in_bom(self):
|
|
"""Test BOM aggregation."""
|
|
parts = self.bob.getRequiredParts()
|
|
|
|
self.assertIn(self.orphan, parts)
|
|
|
|
self.assertTrue(self.bob.check_if_part_in_bom(self.orphan))
|
|
|
|
def test_used_in(self):
|
|
"""Test that the 'used_in_count' attribute is calculated correctly."""
|
|
self.assertEqual(self.bob.used_in_count, 1)
|
|
self.assertEqual(self.orphan.used_in_count, 1)
|
|
|
|
def test_self_reference(self):
|
|
"""Test that we get an appropriate error when we create a BomItem which points to itself."""
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
# A validation error should be raised here
|
|
item = BomItem.objects.create(part=self.bob, sub_part=self.bob, quantity=7)
|
|
item.clean() # pragma: no cover
|
|
|
|
def test_integer_quantity(self):
|
|
"""Test integer validation for BomItem."""
|
|
p = Part.objects.create(
|
|
name='test', description='part description', component=True, trackable=True
|
|
)
|
|
|
|
# Creation of a BOMItem with a non-integer quantity of a trackable Part should fail
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItem.objects.create(part=self.bob, sub_part=p, quantity=21.7)
|
|
|
|
# But with an integer quantity, should be fine
|
|
BomItem.objects.create(part=self.bob, sub_part=p, quantity=21)
|
|
|
|
def test_overage(self):
|
|
"""Test that BOM line overages are calculated correctly."""
|
|
item = BomItem.objects.get(part=100, sub_part=50)
|
|
|
|
q = 300
|
|
|
|
item.quantity = q
|
|
|
|
# Test empty overage
|
|
n = item.get_overage_quantity(q)
|
|
self.assertEqual(n, 0)
|
|
|
|
# Test improper overage
|
|
item.overage = 'asf234?'
|
|
n = item.get_overage_quantity(q)
|
|
self.assertEqual(n, 0)
|
|
|
|
# Test absolute overage
|
|
item.overage = '3'
|
|
n = item.get_overage_quantity(q)
|
|
self.assertEqual(n, 3)
|
|
|
|
# Test percentage-based overage
|
|
item.overage = '5.0 % '
|
|
n = item.get_overage_quantity(q)
|
|
self.assertEqual(n, 15)
|
|
|
|
# Calculate total required quantity
|
|
# Quantity = 300 (+ 5%)
|
|
# Get quantity required to build B = 10
|
|
# Q * B = 3000 + 5% = 3150
|
|
n = item.get_required_quantity(10)
|
|
|
|
self.assertEqual(n, 3150)
|
|
|
|
def test_item_hash(self):
|
|
"""Test BOM item hash encoding."""
|
|
item = BomItem.objects.get(part=100, sub_part=50)
|
|
|
|
h1 = item.get_item_hash()
|
|
|
|
# Change data - the hash must change
|
|
item.quantity += 1
|
|
|
|
h2 = item.get_item_hash()
|
|
|
|
item.validate_hash()
|
|
|
|
self.assertNotEqual(h1, h2)
|
|
|
|
def test_pricing(self):
|
|
"""Test BOM pricing."""
|
|
self.bob.get_price(1)
|
|
self.assertEqual(
|
|
self.bob.get_bom_price_range(1, internal=True),
|
|
(Decimal(29.5), Decimal(89.5)),
|
|
)
|
|
# remove internal price for R_2K2_0805
|
|
self.r1.internal_price_breaks.delete()
|
|
self.assertEqual(
|
|
self.bob.get_bom_price_range(1, internal=True),
|
|
(Decimal(27.5), Decimal(87.5)),
|
|
)
|
|
|
|
def test_substitutes(self):
|
|
"""Tests for BOM item substitutes."""
|
|
# We will make some substitute parts for the "orphan" part
|
|
bom_item = BomItem.objects.get(part=self.bob, sub_part=self.orphan)
|
|
|
|
# No substitute parts available
|
|
self.assertEqual(bom_item.substitutes.count(), 0)
|
|
|
|
subs = []
|
|
|
|
for ii in range(5):
|
|
# Create a new part
|
|
sub_part = Part.objects.create(
|
|
name=f'Orphan {ii}',
|
|
description='A substitute part for the orphan part',
|
|
component=True,
|
|
is_template=False,
|
|
assembly=False,
|
|
)
|
|
|
|
subs.append(sub_part)
|
|
|
|
# Link it as a substitute part
|
|
BomItemSubstitute.objects.create(bom_item=bom_item, part=sub_part)
|
|
|
|
# Try to link it again (this should fail as it is a duplicate substitute)
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
with transaction.atomic():
|
|
BomItemSubstitute.objects.create(bom_item=bom_item, part=sub_part)
|
|
|
|
# There should be now 5 substitute parts available
|
|
self.assertEqual(bom_item.substitutes.count(), 5)
|
|
|
|
# Try to create a substitute which points to the same sub-part (should fail)
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItemSubstitute.objects.create(bom_item=bom_item, part=self.orphan)
|
|
|
|
# Remove one substitute part
|
|
bom_item.substitutes.last().delete()
|
|
|
|
self.assertEqual(bom_item.substitutes.count(), 4)
|
|
|
|
for sub in subs:
|
|
sub.delete()
|
|
|
|
# The substitution links should have been automatically removed
|
|
self.assertEqual(bom_item.substitutes.count(), 0)
|
|
|
|
def test_consumable(self):
|
|
"""Tests for the 'consumable' BomItem field."""
|
|
# Create an assembly part
|
|
assembly = Part.objects.create(
|
|
name='An assembly', description='Made with parts', assembly=True
|
|
)
|
|
|
|
# No BOM information initially
|
|
self.assertEqual(assembly.can_build, 0)
|
|
|
|
# Create some component items
|
|
c1 = Part.objects.create(
|
|
name='C1', description='Part C1 - this is just the part description'
|
|
)
|
|
c2 = Part.objects.create(
|
|
name='C2', description='Part C2 - this is just the part description'
|
|
)
|
|
c3 = Part.objects.create(
|
|
name='C3', description='Part C3 - this is just the part description'
|
|
)
|
|
c4 = Part.objects.create(
|
|
name='C4', description='Part C4 - this is just the part description'
|
|
)
|
|
|
|
for p in [c1, c2, c3, c4]:
|
|
# Ensure we have stock
|
|
stock.models.StockItem.objects.create(part=p, quantity=1000)
|
|
|
|
# Create some BOM items
|
|
BomItem.objects.create(part=assembly, sub_part=c1, quantity=10)
|
|
|
|
self.assertEqual(assembly.can_build, 100)
|
|
|
|
BomItem.objects.create(part=assembly, sub_part=c2, quantity=50, consumable=True)
|
|
|
|
# A 'consumable' BomItem does not alter the can_build calculation
|
|
self.assertEqual(assembly.can_build, 100)
|
|
|
|
BomItem.objects.create(part=assembly, sub_part=c3, quantity=50)
|
|
|
|
self.assertEqual(assembly.can_build, 20)
|
|
|
|
def test_metadata(self):
|
|
"""Unit tests for the metadata field."""
|
|
for model in [BomItem]:
|
|
p = model.objects.first()
|
|
|
|
self.assertIsNone(p.get_metadata('test'))
|
|
self.assertEqual(p.get_metadata('test', backup_value=123), 123)
|
|
|
|
# Test update via the set_metadata() method
|
|
p.set_metadata('test', 3)
|
|
self.assertEqual(p.get_metadata('test'), 3)
|
|
|
|
for k in ['apple', 'banana', 'carrot', 'carrot', 'banana']:
|
|
p.set_metadata(k, k)
|
|
|
|
self.assertEqual(len(p.metadata.keys()), 4)
|
|
|
|
def test_invalid_bom(self):
|
|
"""Test that ValidationError is correctly raised for an invalid BOM item."""
|
|
# First test: A BOM item which points to itself
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItem.objects.create(part=self.bob, sub_part=self.bob, quantity=1)
|
|
|
|
# Second test: A recursive BOM
|
|
part_a = Part.objects.create(
|
|
name='Part A',
|
|
description='A part which is called A',
|
|
assembly=True,
|
|
is_template=True,
|
|
component=True,
|
|
)
|
|
part_b = Part.objects.create(
|
|
name='Part B',
|
|
description='A part which is called B',
|
|
assembly=True,
|
|
component=True,
|
|
)
|
|
part_c = Part.objects.create(
|
|
name='Part C',
|
|
description='A part which is called C',
|
|
assembly=True,
|
|
component=True,
|
|
)
|
|
|
|
BomItem.objects.create(part=part_a, sub_part=part_b, quantity=10)
|
|
BomItem.objects.create(part=part_b, sub_part=part_c, quantity=10)
|
|
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItem.objects.create(part=part_c, sub_part=part_a, quantity=10)
|
|
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItem.objects.create(part=part_c, sub_part=part_b, quantity=10)
|
|
|
|
# Third test: A recursive BOM with a variant part
|
|
part_v = Part.objects.create(
|
|
name='Part V',
|
|
description='A part which is called V',
|
|
variant_of=part_a,
|
|
assembly=True,
|
|
component=True,
|
|
)
|
|
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItem.objects.create(part=part_a, sub_part=part_v, quantity=10)
|
|
|
|
with self.assertRaises(django_exceptions.ValidationError):
|
|
BomItem.objects.create(part=part_v, sub_part=part_a, quantity=10)
|