aa-killtracker 0.12.1__tar.gz → 0.14.0__tar.gz

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: aa-killtracker
-Version: 0.12.1
+Version: 0.14.0
 Summary: "An app for running killmail trackers with Alliance Auth and Discord.
 Author-email: Erik Kalkoken <kalkoken87@gmail.com>
 Requires-Python: >=3.8
@@ -8,6 +8,7 @@ Description-Content-Type: text/markdown
 Classifier: Environment :: Web Environment
 Classifier: Framework :: Django
 Classifier: Framework :: Django :: 4.0
+Classifier: Framework :: Django :: 4.2
 Classifier: Intended Audience :: End Users/Desktop
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Operating System :: OS Independent
@@ -26,7 +27,7 @@ Requires-Dist: django-eveuniverse>=1.3
 Requires-Dist: redis-simple-mq>=0.5
 Project-URL: Home, https://gitlab.com/ErikKalkoken/aa-killtracker
 
-# killtracker
+# Killtracker
 
 An app for running killmail trackers with Alliance Auth and Discord.
 
@@ -99,19 +100,45 @@ pip install aa-killtracker
 
 ### Step 3 - Configure settings
 
-Configure your Auth settings (`local.py`) as follows:
+Configure your AA settings (`local.py`) as follows:
 
-- Add `'killtracker'` to `INSTALLED_APPS`
-- Add below lines to your settings file:
+### Installed apps
+
+Add `'killtracker'` to `INSTALLED_APPS`.
+
+### Killtracker configuration
+
+Add below lines to your settings file:
 
 ```python
+# aa-killtracker
 CELERYBEAT_SCHEDULE['killtracker_run_killtracker'] = {
     'task': 'killtracker.tasks.run_killtracker',
     'schedule': crontab(minute='*/1'),
 }
+KILLTRACKER_QUEUE_ID = ""  # Put your unique queue ID here
+```
+
+### Queue ID
+
+Please note that the queue ID must be globally unique for all users of the zKillboard API, so choose carefully.
+
+We suggest to use your alliance name or alliance tag (without any spaces and special characters) as queue ID.
+
+We recommend using only characters (upper and lower case) and numbers,
+but no spaces or any special characters when choosing your ID.
+
+Example (don't use this exact example):
+
+```Python
+KILLTRACKER_QUEUE_ID = "Voltron9000"
 ```
 
-- Optional: Add additional settings if you want to change any defaults. See [Settings](#settings) for the full list.
+If you are running multiple instances of Killtracker please choose a different queue ID for each of them.
+
+### Additional settings (optional)
+
+Add additional settings if you want to change any defaults. See [Settings](#settings) for the full list.
 
 ### Step 4 - Finalize installation
 
@@ -212,6 +239,7 @@ Name | Description | Default
 `KILLTRACKER_KILLMAIL_MAX_AGE_FOR_TRACKER`| Ignore killmails that are older than the given number in minutes. Sometimes killmails appear belated on ZKB, this feature ensures they don't create new alerts | `60`
 `KILLTRACKER_MAX_KILLMAILS_PER_RUN`| Maximum number of killmails retrieved from ZKB by task run. This value should be set such that the task that fetches new killmails from ZKB every minute will reliable finish within one minute. To test this run a "Catch all" tracker and see how many killmails your system is capable of processing. Note that you can get that information from the worker's log file. It will look something like this: `Total killmails received from ZKB in 49 secs: 251`   | `250`
 `KILLTRACKER_PURGE_KILLMAILS_AFTER_DAYS`| Killmails older than set number of days will be purged from the database. If you want to keep all killmails set this to 0. Note that this setting is only relevant if you have storing killmails enabled.  | `30`
-`KILLTRACKER_WEBHOOK_SET_AVATAR`| Wether app sets the name and avatar icon of a webhook. When False the webhook will use it's own values as set on the platform  | `True`
+`KILLTRACKER_QUEUE_ID`| Unique ID used to identify this server when fetching killmails from zKillboard. This setting is mandatory.  | ``
 `KILLTRACKER_STORING_KILLMAILS_ENABLED`| If set to true Killtracker will automatically store all received killmails in the local database. This can be useful if you want to run analytics on killmails etc. However, please note that Killtracker itself currently does not use stored killmails in any way.  | `False`
+`KILLTRACKER_WEBHOOK_SET_AVATAR`| Wether app sets the name and avatar icon of a webhook. When False the webhook will use it's own values as set on the platform  | `True`
 

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/README.md

@@ -1,4 +1,4 @@
-# killtracker
+# Killtracker
 
 An app for running killmail trackers with Alliance Auth and Discord.
 
@@ -71,19 +71,45 @@ pip install aa-killtracker
 
 ### Step 3 - Configure settings
 
-Configure your Auth settings (`local.py`) as follows:
+Configure your AA settings (`local.py`) as follows:
 
-- Add `'killtracker'` to `INSTALLED_APPS`
-- Add below lines to your settings file:
+### Installed apps
+
+Add `'killtracker'` to `INSTALLED_APPS`.
+
+### Killtracker configuration
+
+Add below lines to your settings file:
 
 ```python
+# aa-killtracker
 CELERYBEAT_SCHEDULE['killtracker_run_killtracker'] = {
     'task': 'killtracker.tasks.run_killtracker',
     'schedule': crontab(minute='*/1'),
 }
+KILLTRACKER_QUEUE_ID = ""  # Put your unique queue ID here
+```
+
+### Queue ID
+
+Please note that the queue ID must be globally unique for all users of the zKillboard API, so choose carefully.
+
+We suggest to use your alliance name or alliance tag (without any spaces and special characters) as queue ID.
+
+We recommend using only characters (upper and lower case) and numbers,
+but no spaces or any special characters when choosing your ID.
+
+Example (don't use this exact example):
+
+```Python
+KILLTRACKER_QUEUE_ID = "Voltron9000"
 ```
 
-- Optional: Add additional settings if you want to change any defaults. See [Settings](#settings) for the full list.
+If you are running multiple instances of Killtracker please choose a different queue ID for each of them.
+
+### Additional settings (optional)
+
+Add additional settings if you want to change any defaults. See [Settings](#settings) for the full list.
 
 ### Step 4 - Finalize installation
 
@@ -184,5 +210,6 @@ Name | Description | Default
 `KILLTRACKER_KILLMAIL_MAX_AGE_FOR_TRACKER`| Ignore killmails that are older than the given number in minutes. Sometimes killmails appear belated on ZKB, this feature ensures they don't create new alerts | `60`
 `KILLTRACKER_MAX_KILLMAILS_PER_RUN`| Maximum number of killmails retrieved from ZKB by task run. This value should be set such that the task that fetches new killmails from ZKB every minute will reliable finish within one minute. To test this run a "Catch all" tracker and see how many killmails your system is capable of processing. Note that you can get that information from the worker's log file. It will look something like this: `Total killmails received from ZKB in 49 secs: 251`   | `250`
 `KILLTRACKER_PURGE_KILLMAILS_AFTER_DAYS`| Killmails older than set number of days will be purged from the database. If you want to keep all killmails set this to 0. Note that this setting is only relevant if you have storing killmails enabled.  | `30`
-`KILLTRACKER_WEBHOOK_SET_AVATAR`| Wether app sets the name and avatar icon of a webhook. When False the webhook will use it's own values as set on the platform  | `True`
+`KILLTRACKER_QUEUE_ID`| Unique ID used to identify this server when fetching killmails from zKillboard. This setting is mandatory.  | ``
 `KILLTRACKER_STORING_KILLMAILS_ENABLED`| If set to true Killtracker will automatically store all received killmails in the local database. This can be useful if you want to run analytics on killmails etc. However, please note that Killtracker itself currently does not use stored killmails in any way.  | `False`
+`KILLTRACKER_WEBHOOK_SET_AVATAR`| Wether app sets the name and avatar icon of a webhook. When False the webhook will use it's own values as set on the platform  | `True`

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/__init__.py

@@ -3,7 +3,7 @@
 # pylint: disable = invalid-name
 default_app_config = "killtracker.apps.KillmailsConfig"
 
-__version__ = "0.12.1"
+__version__ = "0.14.0"
 __title__ = "Killtracker"
 
 APP_NAME = "aa-killtracker"

aa_killtracker-0.14.0/killtracker/app_settings.py

@@ -0,0 +1,101 @@
+"""Settings for killtracker."""
+
+from app_utils.app_settings import clean_setting
+
+KILLTRACKER_REDISQ_LOCK_TIMEOUT = clean_setting("KILLTRACKER_REDISQ_LOCK_TIMEOUT", 5)
+"""Timeout for lock to ensure atomic access to ZKB RedisQ."""
+
+KILLTRACKER_KILLMAIL_MAX_AGE_FOR_TRACKER = clean_setting(
+    "KILLTRACKER_KILLMAIL_MAX_AGE_FOR_TRACKER", 60
+)
+"""Ignore killmails that are older than the given number in minutes
+sometimes killmails appear belated on ZKB,
+this feature ensures they don't create new alerts.
+"""
+
+KILLTRACKER_MAX_KILLMAILS_PER_RUN = clean_setting(
+    "KILLTRACKER_MAX_KILLMAILS_PER_RUN", 200
+)
+"""Maximum number of killmails retrieved from ZKB by task run."""
+
+KILLTRACKER_PURGE_KILLMAILS_AFTER_DAYS = clean_setting(
+    "KILLTRACKER_PURGE_KILLMAILS_AFTER_DAYS", 30
+)
+"""Killmails older than set number of days will be purged from the database.
+If you want to keep all killmails set this to 0.
+"""
+
+KILLTRACKER_QUEUE_ID = clean_setting("KILLTRACKER_QUEUE_ID", "")
+"""Unique ID used to identify this server when fetching killmails from zKillboard.
+
+Please note that the queue ID must be globally unique for all users of the zKillboard API, so choose carefully.
+
+We recommend using only characters (upper and lower case) and numbers,
+but no spaces or any special characters when choosing your ID.
+
+Example: ``"Voltron9000"`` (don't use this exact example)
+
+We suggest to use your alliance or corporation name (without spaces and special characters) as queue ID.
+
+If you are running multiple instances of Killtracker please choose a different queue ID for each of them.
+
+This setting is mandatory.
+"""
+
+
+KILLTRACKER_STORING_KILLMAILS_ENABLED = clean_setting(
+    "KILLTRACKER_STORING_KILLMAILS_ENABLED", False
+)
+"""Whether killmails retrieved from ZKB are stored in the database."""
+
+KILLTRACKER_WEBHOOK_SET_AVATAR = clean_setting("KILLTRACKER_WEBHOOK_SET_AVATAR", True)
+"""Wether app sets the name and avatar icon of a webhook.
+When False the webhook will use it's own values as set on the platform.
+"""
+
+KILLTRACKER_SHOW_NPC_TYPES = clean_setting("KILLTRACKER_SHOW_NPC_TYPES", True)
+"""Wether NPC types (e.g. Guristas Assaulter) can be selected as attacker types
+when creating trackers.
+"""
+
+
+#####################
+# INTERNAL SETTINGS
+
+KILLTRACKER_REDISQ_TTW = clean_setting("KILLTRACKER_REDISQ_TTW", 5)
+"""Max duration to wait for new killmails from redisq in seconds."""
+
+KILLTRACKER_TASKS_TIMEOUT = clean_setting("KILLTRACKER_TASKS_TIMEOUT", 1_800)
+"""Tasks hard timeout."""
+
+KILLTRACKER_DISCORD_SEND_DELAY = clean_setting(
+    "KILLTRACKER_DISCORD_SEND_DELAY", default_value=2, min_value=1, max_value=900
+)
+"""Delay in seconds between every message sent to Discord
+this needs to be >= 1 to prevent 429 Too Many Request errors.
+"""
+
+KILLTRACKER_GENERATE_MESSAGE_MAX_RETRIES = clean_setting(
+    "KILLTRACKER_GENERATE_MESSAGE_MAX_RETRIES", 3
+)
+"""Maximum retries when generating a message from a killmail."""
+
+KILLTRACKER_GENERATE_MESSAGE_RETRY_COUNTDOWN = clean_setting(
+    "KILLTRACKER_GENERATE_MESSAGE_RETRY_COUNTDOWN", 10
+)
+"""Delay when retrying to generate a message in seconds."""
+
+KILLTRACKER_TASK_OBJECTS_CACHE_TIMEOUT = clean_setting(
+    "KILLTRACKER_TASK_OBJECTS_CACHE_TIMEOUT", 60
+)
+"""Cache duration for objects in tasks in seconds."""
+
+KILLTRACKER_TASK_MINIMUM_RETRY_DELAY = clean_setting(
+    "KILLTRACKER_TASK_MINIMUM_RETRY_DELAY", default_value=0.05
+)
+"""Minimum delay when retrying a task."""
+
+KILLTRACKER_STORAGE_KILLMAILS_LIFETIME = clean_setting(
+    "KILLTRACKER_STORAGE_KILLMAILS_LIFETIME", 3_600 * 1
+)
+"""Max lifetime of killmails in temporary storage in seconds."""

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/core/killmails.py

@@ -8,6 +8,7 @@ from dataclasses import asdict, dataclass
 from datetime import datetime
 from http import HTTPStatus
 from typing import List, Optional, Set
+from urllib.parse import quote_plus
 
 import requests
 from dacite import DaciteError, from_dict
@@ -16,6 +17,7 @@ from simplejson.errors import JSONDecodeError
 
 from django.conf import settings
 from django.core.cache import cache
+from django.core.exceptions import ImproperlyConfigured
 from django.utils.dateparse import parse_datetime
 from eveuniverse.models import EveType
 
@@ -26,6 +28,7 @@ from app_utils.logging import LoggerAddTag
 
 from killtracker import USER_AGENT_TEXT, __title__
 from killtracker.app_settings import (
+    KILLTRACKER_QUEUE_ID,
     KILLTRACKER_REDISQ_LOCK_TIMEOUT,
     KILLTRACKER_REDISQ_TTW,
     KILLTRACKER_STORAGE_KILLMAILS_LIFETIME,
@@ -43,6 +46,8 @@ REQUESTS_TIMEOUT = (5, 30)
 MAIN_MINIMUM_COUNT = 2
 MAIN_MINIMUM_SHARE = 0.25
 
+# TODO: Factor out logic for accessing the API to another module
+
 
 @dataclass
 class _KillmailBase:
@@ -385,15 +390,27 @@ class Killmail(_KillmailBase):
 
         Returns None if no killmail is received.
         """
-        logger.info("Trying to fetch killmail from ZKB RedisQ...")
+        if not KILLTRACKER_QUEUE_ID:
+            raise ImproperlyConfigured(
+                "You need to define a queue ID in your settings."
+            )
+
+        if "," in KILLTRACKER_QUEUE_ID:
+            raise ImproperlyConfigured("A queue ID must not contains commas.")
+
         redis = get_redis_client()
+        params = {
+            "queueID": quote_plus(KILLTRACKER_QUEUE_ID),
+            "ttw": KILLTRACKER_REDISQ_TTW,
+        }
         try:
+            logger.info("Trying to fetch killmail from ZKB RedisQ...")
             with redis.lock(
                 cls.lock_key(), blocking_timeout=KILLTRACKER_REDISQ_LOCK_TIMEOUT
             ):
                 response = requests.get(
                     ZKB_REDISQ_URL,
-                    params={"ttw": KILLTRACKER_REDISQ_TTW},
+                    params=params,
                     timeout=REQUESTS_TIMEOUT,
                     headers={"User-Agent": USER_AGENT_TEXT},
                 )
@@ -407,7 +424,9 @@ class Killmail(_KillmailBase):
         if response.status_code == HTTPStatus.TOO_MANY_REQUESTS:
             logger.error("429 Client Error: Too many requests: %s", response.text)
             return None
+
         response.raise_for_status()
+
         try:
             data = response.json()
         except JSONDecodeError:

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/core/test_discord_messages_1.py

@@ -3,15 +3,14 @@ import dhooks_lite
 from app_utils.testing import NoSocketsTestCase
 
 from killtracker.core import discord_messages
-
-from ..testdata.factories import (
+from killtracker.tests.testdata.factories import (
     EveEntityVariant,
     KillmailFactory,
     TrackerFactory,
     random_eve_entity,
 )
-from ..testdata.helpers import load_eve_entities
-from ..testdata.load_eveuniverse import load_eveuniverse
+from killtracker.tests.testdata.helpers import load_eve_entities
+from killtracker.tests.testdata.load_eveuniverse import load_eveuniverse
 
 
 class TestCreateEmbed(NoSocketsTestCase):

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/core/test_discord_messages_2.py

@@ -10,9 +10,8 @@ from app_utils.django import app_labels
 
 from killtracker.core.killmails import Killmail
 from killtracker.models import Tracker
-
-from ..testdata.factories import TrackerFactory
-from ..testdata.helpers import LoadTestDataMixin, load_killmail
+from killtracker.tests.testdata.factories import TrackerFactory
+from killtracker.tests.testdata.helpers import LoadTestDataMixin, load_killmail
 
 DISCORD_MESSAGES_PATH = "killtracker.core.discord_messages"
 

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/core/test_killmails.py

@@ -6,6 +6,7 @@ import requests_mock
 from redis.exceptions import LockError
 
 from django.core.cache import cache
+from django.core.exceptions import ImproperlyConfigured
 from django.test import TestCase
 from django.utils.dateparse import parse_datetime
 from django.utils.timezone import now
@@ -20,18 +21,19 @@ from killtracker.core.killmails import (
     _EntityCount,
 )
 from killtracker.exceptions import KillmailDoesNotExist
-
-from .. import CacheStub
-from ..testdata.factories import KillmailFactory
-from ..testdata.helpers import killmails_data, load_killmail
+from killtracker.tests import CacheStub
+from killtracker.tests.testdata.factories import KillmailFactory
+from killtracker.tests.testdata.helpers import killmails_data, load_killmail
 
 MODULE_PATH = "killtracker.core.killmails"
 unittest.util._MAX_LENGTH = 1000
+requests_mock.mock.case_sensitive = True
 
 
 @requests_mock.Mocker()
 @patch(MODULE_PATH + ".get_redis_client")
 class TestCreateFromZkbRedisq(NoSocketsTestCase):
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_return_killmail(self, requests_mocker, mock_redis):
         # given
         requests_mocker.register_uri(
@@ -71,6 +73,7 @@ class TestCreateFromZkbRedisq(NoSocketsTestCase):
         self.assertFalse(killmail.zkb.is_solo)
         self.assertFalse(killmail.zkb.is_awox)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_return_none_when_zkb_returns_empty_package(
         self, requests_mocker, mock_redis
     ):
@@ -83,6 +86,7 @@ class TestCreateFromZkbRedisq(NoSocketsTestCase):
         # then
         self.assertIsNone(killmail)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_handle_zkb_data_has_no_solar_system(
         self, requests_mocker, mock_redis
     ):
@@ -98,6 +102,7 @@ class TestCreateFromZkbRedisq(NoSocketsTestCase):
         # then
         self.assertIsNotNone(killmail)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_return_none_when_zkb_returns_429_error(
         self, requests_mocker, mock_redis
     ):
@@ -110,6 +115,7 @@ class TestCreateFromZkbRedisq(NoSocketsTestCase):
         # then
         self.assertIsNone(killmail)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_return_none_when_zkb_returns_general_error(
         self, requests_mocker, mock_redis
     ):
@@ -127,6 +133,7 @@ You can only have one request to listen.php in flight at any time, otherwise you
         # then
         self.assertIsNone(killmail)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_return_none_when_zkb_does_not_return_json(
         self, requests_mocker, mock_redis
     ):
@@ -139,6 +146,7 @@ You can only have one request to listen.php in flight at any time, otherwise you
         # then
         self.assertIsNone(killmail)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "dummy")
     def test_should_return_none_if_lock_not_acquired(self, requests_mocker, mock_redis):
         # given
         mock_redis.return_value.lock.side_effect = LockError
@@ -147,6 +155,56 @@ You can only have one request to listen.php in flight at any time, otherwise you
         # then
         self.assertIsNone(killmail)
 
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "Voltron9000")
+    def test_should_have_queue_id_in_request(self, requests_mocker, mock_redis):
+        # given
+        requests_mocker.register_uri(
+            "GET", ZKB_REDISQ_URL, status_code=200, json={"package": None}
+        )
+        # when
+        Killmail.create_from_zkb_redisq()
+        # then
+        qs = requests_mocker.last_request.qs
+        self.assertIn("queueID", qs)
+        queue_id = qs["queueID"]
+        self.assertEqual(len(queue_id), 1)
+        self.assertEqual(queue_id[0], "Voltron9000")
+
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "")
+    def test_should_abort_when_no_queue_id_defined(self, requests_mocker, mock_redis):
+        # given
+        requests_mocker.register_uri(
+            "GET", ZKB_REDISQ_URL, status_code=200, json={"package": None}
+        )
+        # when/then
+        with self.assertRaises(ImproperlyConfigured):
+            Killmail.create_from_zkb_redisq()
+
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "Möchtegern")
+    def test_should_urlize_queue_ids(self, requests_mocker, mock_redis):
+        # given
+        requests_mocker.register_uri(
+            "GET", ZKB_REDISQ_URL, status_code=200, json={"package": None}
+        )
+        # when
+        Killmail.create_from_zkb_redisq()
+        # then
+        qs = requests_mocker.last_request.qs
+        self.assertIn("queueID", qs)
+        queue_id = qs["queueID"]
+        self.assertEqual(len(queue_id), 1)
+        self.assertEqual(queue_id[0], "M%C3%B6chtegern")
+
+    @patch(MODULE_PATH + ".KILLTRACKER_QUEUE_ID", "alpha,bravo")
+    def test_should_not_accept_list_for_queue_id(self, requests_mocker, mock_redis):
+        # given
+        requests_mocker.register_uri(
+            "GET", ZKB_REDISQ_URL, status_code=200, json={"package": None}
+        )
+        # when/then
+        with self.assertRaises(ImproperlyConfigured):
+            Killmail.create_from_zkb_redisq()
+
 
 class TestKillmailSerialization(NoSocketsTestCase):
     def test_dict_serialization(self):

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/models/test_killmails.py

@@ -7,15 +7,17 @@ from eveuniverse.models import EveEntity
 from app_utils.testing import NoSocketsTestCase
 
 from killtracker.models import EveKillmail
-
-from ..testdata.factories import EveKillmailAttackerFactory, EveKillmailFactory
-from ..testdata.helpers import (
+from killtracker.tests.testdata.factories import (
+    EveKillmailAttackerFactory,
+    EveKillmailFactory,
+)
+from killtracker.tests.testdata.helpers import (
     LoadTestDataMixin,
     load_eve_entities,
     load_eve_killmails,
     load_killmail,
 )
-from ..testdata.load_eveuniverse import load_eveuniverse
+from killtracker.tests.testdata.load_eveuniverse import load_eveuniverse
 
 MODELS_PATH = "killtracker.models.killmails"
 

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/models/test_trackers_1.py

@@ -22,14 +22,13 @@ from app_utils.testing import NoSocketsTestCase, add_character_to_user_2
 from killtracker.constants import EveGroupId
 from killtracker.core.killmails import Killmail, _EntityCount
 from killtracker.models import Tracker
-
-from ..testdata.factories import (
+from killtracker.tests.testdata.factories import (
     KillmailAttackerFactory,
     KillmailFactory,
     KillmailVictimFactory,
     TrackerFactory,
 )
-from ..testdata.helpers import LoadTestDataMixin, load_killmail
+from killtracker.tests.testdata.helpers import LoadTestDataMixin, load_killmail
 
 MODELS_PATH = "killtracker.models"
 

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/models/test_trackers_2.py

@@ -9,9 +9,8 @@ from eveuniverse.models import EveConstellation, EveRegion, EveSolarSystem, EveT
 from app_utils.testing import NoSocketsTestCase
 
 from killtracker.core.killmails import Killmail
-
-from ..testdata.factories import TrackerFactory
-from ..testdata.helpers import LoadTestDataMixin, load_killmail
+from killtracker.tests.testdata.factories import TrackerFactory
+from killtracker.tests.testdata.helpers import LoadTestDataMixin, load_killmail
 
 MODELS_PATH = "killtracker.models.trackers"
 

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/test_admin.py

@@ -62,7 +62,7 @@ class TestTrackerValidations(LoadTestDataMixin, WebTest):
         # user tries to add new notification rule
         add_page = self.app.get(self.url_add)
         self.assertEqual(add_page.status_code, 200)
-        form = add_page.form
+        form = add_page.forms["tracker_form"]
         form["name"] = "Test Tracker"
         form["webhook"] = self.webhook_1.pk
         return form

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/killtracker/tests/test_integration.py

@@ -18,6 +18,7 @@ PACKAGE_PATH = "killtracker"
 
 @patch("celery.app.task.Context.called_directly", False)  # make retry work with eager
 @override_settings(CELERY_ALWAYS_EAGER=True)
+@patch(PACKAGE_PATH + ".core.killmails.KILLTRACKER_QUEUE_ID", "dummy")
 @patch(PACKAGE_PATH + ".tasks.is_esi_online", lambda: True)
 @patch(PACKAGE_PATH + ".models.webhooks.dhooks_lite.Webhook.execute", spec=True)
 @requests_mock.Mocker()

{aa_killtracker-0.12.1 → aa_killtracker-0.14.0}/pyproject.toml

@@ -13,6 +13,7 @@ classifiers = [
     "Environment :: Web Environment",
     "Framework :: Django",
     "Framework :: Django :: 4.0",
+    "Framework :: Django :: 4.2",
     "Intended Audience :: End Users/Desktop",
     "License :: OSI Approved :: MIT License",
     "Operating System :: OS Independent",

aa_killtracker-0.12.1/killtracker/app_settings.py

@@ -1,78 +0,0 @@
-"""Settings for killtracker."""
-
-from app_utils.app_settings import clean_setting
-
-# Timeout for lock to ensure atomic access to ZKB RedisQ
-KILLTRACKER_REDISQ_LOCK_TIMEOUT = clean_setting("KILLTRACKER_REDISQ_LOCK_TIMEOUT", 5)
-
-# ignore killmails that are older than the given number in minutes
-# sometimes killmails appear belated on ZKB,
-# this feature ensures they don't create new alerts
-KILLTRACKER_KILLMAIL_MAX_AGE_FOR_TRACKER = clean_setting(
-    "KILLTRACKER_KILLMAIL_MAX_AGE_FOR_TRACKER", 60
-)
-
-# Maximum number of killmails retrieved from ZKB by task run
-KILLTRACKER_MAX_KILLMAILS_PER_RUN = clean_setting(
-    "KILLTRACKER_MAX_KILLMAILS_PER_RUN", 200
-)
-
-# Killmails older than set number of days will be purged from the database.
-# If you want to keep all killmails set this to 0.
-KILLTRACKER_PURGE_KILLMAILS_AFTER_DAYS = clean_setting(
-    "KILLTRACKER_PURGE_KILLMAILS_AFTER_DAYS", 30
-)
-
-# whether killmails retrieved from ZKB are stored in the database
-KILLTRACKER_STORING_KILLMAILS_ENABLED = clean_setting(
-    "KILLTRACKER_STORING_KILLMAILS_ENABLED", False
-)
-
-# Wether app sets the name and avatar icon of a webhook.
-# When False the webhook will use it's own values as set on the platform
-KILLTRACKER_WEBHOOK_SET_AVATAR = clean_setting("KILLTRACKER_WEBHOOK_SET_AVATAR", True)
-
-
-# Wether NPC types (e.g. Guristas Assaulter) can be selected as attacker types
-# when creating trackers
-KILLTRACKER_SHOW_NPC_TYPES = clean_setting("KILLTRACKER_SHOW_NPC_TYPES", True)
-
-#####################
-# INTERNAL SETTINGS
-
-# Max duration to wait for new killmails from redisq in seconds
-KILLTRACKER_REDISQ_TTW = clean_setting("KILLTRACKER_REDISQ_TTW", 5)
-
-# Tasks hard timeout
-KILLTRACKER_TASKS_TIMEOUT = clean_setting("KILLTRACKER_TASKS_TIMEOUT", 1_800)
-
-# delay in seconds between every message sent to Discord
-# this needs to be >= 1 to prevent 429 Too Many Request errors
-KILLTRACKER_DISCORD_SEND_DELAY = clean_setting(
-    "KILLTRACKER_DISCORD_SEND_DELAY", default_value=2, min_value=1, max_value=900
-)
-
-# Maximum retries when generating a message from a killmail
-KILLTRACKER_GENERATE_MESSAGE_MAX_RETRIES = clean_setting(
-    "KILLTRACKER_GENERATE_MESSAGE_MAX_RETRIES", 3
-)
-
-# Delay when retrying to generate a message in seconds
-KILLTRACKER_GENERATE_MESSAGE_RETRY_COUNTDOWN = clean_setting(
-    "KILLTRACKER_GENERATE_MESSAGE_RETRY_COUNTDOWN", 10
-)
-
-# Cache duration for objects in tasks in seconds
-KILLTRACKER_TASK_OBJECTS_CACHE_TIMEOUT = clean_setting(
-    "KILLTRACKER_TASK_OBJECTS_CACHE_TIMEOUT", 60
-)
-
-# Minimum delay when retrying a task
-KILLTRACKER_TASK_MINIMUM_RETRY_DELAY = clean_setting(
-    "KILLTRACKER_TASK_MINIMUM_RETRY_DELAY", default_value=0.05
-)
-
-# Max lifetime of killmails in temporary storage in seconds
-KILLTRACKER_STORAGE_KILLMAILS_LIFETIME = clean_setting(
-    "KILLTRACKER_STORAGE_KILLMAILS_LIFETIME", 3_600 * 1
-)