Skip to content

Conversation

@PeeachPie
Copy link
Collaborator

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
codeplag
   __init__.py33316%5–6, 8–14, 16–17, 19–20, 22–34, 40–43, 45
   __main__.py330%1, 3–4
   codeplagcli.py101892%60–61, 63, 428, 439, 442, 448, 460
   config.py390100% 
   consts.py430100% 
   display.py1284961%45, 71, 99–102, 104, 116, 118–119, 121–124, 126–127, 129, 135, 137–138, 140–143, 145–146, 148, 150, 154, 170–172, 180–181, 188–189, 191, 196–197, 199–200, 202, 206–210, 214–215
   featurescache.py29679%31–32, 34, 37–38, 40
   getfeatures.py1137038%97–100, 102–104, 106–108, 113–116, 118–120, 125, 128–129, 135–137, 163–168, 170, 172, 195–199, 201–204, 207–210, 212–213, 215, 217–218, 220, 240–244, 246–248, 250–251, 254–257, 259–260, 262, 264–265, 267
   logger.py613345%28–30, 33, 35, 40–41, 45, 49–51, 58, 62–64, 66, 70–72, 74–77, 79, 85–90, 94–95, 104
   reporters.py701085%40, 43, 57–58, 74, 76, 88, 92–93, 98
   translate.py5180%10
   types.py1430100% 
   utils.py44440%1–2, 4, 6, 11–12, 15–17, 20–23, 25–28, 30–35, 37–41, 43–44, 53–54, 56–57, 59–66, 70, 76
codeplag/algorithms
   __init__.py00100% 
   compare.py240100% 
   featurebased.py129397%36, 84, 102
   stringbased.py560100% 
   tokenbased.py45393%118, 137, 170
codeplag/cplag
   __init__.py00100% 
   const.py90100% 
   tree.py47491%33–36
   utils.py804741%22–24, 41–42, 61–62, 64–66, 68–69, 71–75, 77–80, 82, 93, 102, 104–105, 107–113, 116, 119–124, 126, 129–130, 132–133, 136, 142
codeplag/db
   __init__.py00100% 
   mongo.py1131884%62–64, 78–83, 85, 92–95, 121–122, 181–182
codeplag/handlers
   __init__.py00100% 
   check.py20815326%87–90, 92, 94–95, 97–102, 105, 109–116, 118–122, 124–125, 127, 129, 136–137, 149–150, 153, 155–159, 176–181, 183–185, 187–190, 196–197, 203–207, 216–220, 222–225, 230–239, 242–243, 252, 261–265, 270–281, 286–289, 292–293, 303–305, 307–311, 314, 319–329, 338, 341–343, 346–349, 351–352, 354, 362, 369–373, 379–380, 388–389, 404, 407, 445–448
   report.py2159953%84–94, 97–99, 108–111, 120–121, 123, 127, 143, 149–151, 250–251, 301–303, 306, 311–315, 319–321, 326–330, 333–335, 337, 349–352, 354–358, 361, 365–370, 394–398, 400–405, 432–435, 437–440, 442–447, 449, 456–458, 460–463, 472
   settings.py22220%1–2, 4–5, 7–8, 11–15, 20, 23–30, 32, 34
codeplag/pyplag
   __init__.py00100% 
   astwalkers.py65789%25–30, 38
   const.py70100% 
   utils.py945640%26–27, 34–35, 42–43, 51–52, 57–59, 73–74, 76–77, 83, 85–89, 106–107, 109–111, 113–114, 116–119, 121–124, 126, 137, 146, 148–149, 151–157, 159, 163, 166–167, 169–170, 173, 179
webparsers
   __init__.py00100% 
   async_github_parser.py1751750%1–5, 7–12, 14, 24, 27, 52–53, 55, 57, 59–60, 62, 64, 66–67, 69, 71, 79–80, 82, 84–86, 93–95, 97, 99, 104–109, 111, 114–116, 118–124, 126–127, 129–138, 140, 142–146, 148–151, 156–158, 168–169, 171, 173, 175–176, 178, 180, 183, 187, 189–193, 195–200, 209–210, 212, 214, 216, 222, 226–227, 229, 232, 243, 245, 247, 256, 259–266, 274–276, 281, 283, 285, 291, 293, 296–300, 302–303, 305–306, 309, 319–320, 327, 329–330, 338, 344–346, 349–351, 354, 362–363, 368, 370, 374, 380, 382, 385–389, 391, 401–406, 408, 411–412
   github_parser.py1842685%37, 56–58, 64, 74, 77–80, 88–91, 122, 151, 251, 260–262, 347–349, 360, 363–364
   types.py630100% 
TOTAL234886863% 

@Artanias Artanias added the Refactoring Changes made to the code to improve readability, speed of execution, and logic between parts. label Nov 8, 2025
@Artanias
Copy link
Collaborator

Artanias commented Nov 8, 2025

Поднимите, пожалуйста, версию до 6.0.1 0.6.1 тут.

@Artanias
Copy link
Collaborator

Artanias commented Nov 8, 2025

Также стоит не забыть после изменений обновить переводы (make translate-update) и пройтись по переводам.

@Artanias Artanias added this to the v0.6.0 milestone Nov 8, 2025
@PeeachPie
Copy link
Collaborator Author

PeeachPie commented Nov 8, 2025

Поднимите, пожалуйста, версию до 6.0.1 тут.

меня немного смущает, что там 0.6.0, точно ли нам нужно 6.0.1, а не 0.6.1

@Artanias
Copy link
Collaborator

Artanias commented Nov 8, 2025

Поднимите, пожалуйста, версию до 6.0.1 тут.

меня немного смущает, что там 0.6.0, точно ли нам нужно 6.0.1, а не 0.6.1

Верно, 0.6.1, ошибся.

@PeeachPie
Copy link
Collaborator Author

PeeachPie commented Nov 8, 2025

Асинхронную версию парсера пока не трогал, когда обычная будет принята - поменяю. Ридми тоже не трогал. Переводы еще раз обновлю перед мержем.

@Artanias
Copy link
Collaborator

Artanias commented Nov 8, 2025

Обновите, пожалуйста, демо примеры в README.md. Там необходимо заменить старые опции новой.

@Artanias Artanias changed the title github-url added, github-project-folder github-files removed refactor: github-url added instead of github-project-folder and github-files options Nov 8, 2025
@PeeachPie
Copy link
Collaborator Author

@Artanias, добрый вечер. Некоторое время назад удалил все контейнеры с системы. Не уверен, что следующая проблема с этим связана, по теперь при выполнении команды $ make docker-image во время тестирования стали появляться ошибки вида:

ERROR test/unit/codeplag/db/test_mongo.py::TestMongoDBInfrastructure::test_mongodb_connection - AssertionError: Invalid MONGO_HOST environment ''.
assert ''

Во время команды создаются контейнеры:

CONTAINER ID   IMAGE                             COMMAND                  CREATED         STATUS        PORTS       NAMES
7235ffc5c237   codeplag-test-ubuntu24.04:0.6.1   "/bin/sh -c 'make te…"   2 seconds ago   Up 1 second               determined_shockley
bef0a2e96bbc   mongo:8.0.9                       "docker-entrypoint.s…"   2 seconds ago   Up 1 second   27017/tcp   mongo-codeplag-test

Причем я проверил и для ветки main. Ошибка та же. Подскажите, пожалуйста, с чем это может быть связанно?

@Artanias
Copy link
Collaborator

Artanias commented Nov 29, 2025

MONGO_HOST

Добрый вечер, @PeeachPie, проверил на текущей ветке, образ собирается, все юнит тесты проходят. Предварительно выполнил make clean-all, чтобы зачистить артефакты, т.к. бывает без этого не пересобрать образ локально.

Похоже некорректно отрабатывает тут логика. Надо будет похоже рефакторить. А какая версия docker используется?

Т.е. посмотреть бы что в секции NetworkSettings и есть ли она вообще у вас в момент запуска контейнера с mongo и вызова docker inspect mongo-codeplag-test.

@PeeachPie
Copy link
Collaborator Author

Предварительно выполнил make clean-all, чтобы зачистить артефакты, т.к. бывает без этого не пересобрать образ локально.

К сожалению это не дало никакого эффекта.

А какая версия docker используется?

[peach@pie ~]$ docker --version
Docker version 29.1.1, build 0aedba58c2
[peach@pie ~]$ docker-compose --version
Docker Compose version 2.40.3

Т.е. посмотреть бы что в секции NetworkSettings и есть ли она вообще у вас в момент запуска контейнера с mongo и вызова docker inspect mongo-codeplag-test.

"NetworkSettings": {
            "SandboxID": "2f5eb4d3e34180a9ca327eb3c4cccb8dfd95642be721fce04657e590c2815e6b",
            "SandboxKey": "/var/run/docker/netns/2f5eb4d3e341",
            "Ports": {
                "27017/tcp": null
            },
            "Networks": {
                "bridge": {
                    "IPAMConfig": null,
                    "Links": null,
                    "Aliases": null,
                    "DriverOpts": null,
                    "GwPriority": 0,
                    "NetworkID": "0f85f641f522f5aa61e0f7b7f2c929272ccd771f4d1a2118270c4100fbc5a586",
                    "EndpointID": "f18ef5631bd6f778c77fd435737a28ee3ee1be2f43db9cae81e22abf3560ea28",
                    "Gateway": "172.17.0.1",
                    "IPAddress": "172.17.0.2",
                    "MacAddress": "d6:d4:ac:dd:6c:3b",
                    "IPPrefixLen": 16,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "DNSNames": null
                }
            }
        }

@Artanias
Copy link
Collaborator

Artanias commented Nov 30, 2025

"NetworkSettings": {
            "SandboxID": "2f5eb4d3e34180a9ca327eb3c4cccb8dfd95642be721fce04657e590c2815e6b",
            "SandboxKey": "/var/run/docker/netns/2f5eb4d3e341",
            "Ports": {
                "27017/tcp": null
            },
            "Networks": {
                "bridge": {
                    "IPAMConfig": null,
                    "Links": null,
                    "Aliases": null,
                    "DriverOpts": null,
                    "GwPriority": 0,
                    "NetworkID": "0f85f641f522f5aa61e0f7b7f2c929272ccd771f4d1a2118270c4100fbc5a586",
                    "EndpointID": "f18ef5631bd6f778c77fd435737a28ee3ee1be2f43db9cae81e22abf3560ea28",
                    "Gateway": "172.17.0.1",
                    "IPAddress": "172.17.0.2",
                    "MacAddress": "d6:d4:ac:dd:6c:3b",
                    "IPPrefixLen": 16,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "DNSNames": null
                }
            }
        }

@PeeachPie, т.е. похоже адрес переместился глубже в новой версии (NetworkSettings.Networks.bridge.IPAddress).

А если заменить там на извлечение '{{range.NetworkSettings.Networks}}{{.IPAddress}}{{end}}'. На 65 и 79 строках, заработает?

Создал PR на этот фикс https://github.com/OSLL/code-plagiarism/pull/247/files.

@PeeachPie
Copy link
Collaborator Author

А если заменить там на извлечение '{{range.NetworkSettings.Networks}}{{.IPAddress}}{{end}}'. На 65 и 79 строках, заработает?

@Artanias, Да! Спасибо

@PeeachPie PeeachPie requested a review from Artanias December 4, 2025 08:32
@Artanias Artanias merged commit d1d8c51 into main Dec 5, 2025
4 checks passed
@Artanias Artanias deleted the github_url_arg branch December 5, 2025 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactoring Changes made to the code to improve readability, speed of execution, and logic between parts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unify paths and urls into less options for future scalable and easy usability.

3 participants