Rtop 59 plugin driver layer unit tests

[marconetsf]

Intro

A funcionalidade dos plugin drivers depende de que suas configurações e inicializações aconteçam com zero problemas.
Esta branch possui a implementação de funções que compoem a solução. É subdividida em duas partes. A primeira são os testes de configuração (Já que os plugins possuem um arquivo plugins.conf que ditam onde se encontram os requisitos necessários para rodar os plugins.

Testes de configuração

• Parsing de arquivo de configuração válido: Verifica se o arquivo de configuração com plugins válidos é lido e interpretado corretamente.
• Parsing de arquivo com mais plugins do que o limite: Testa se o parser respeita o limite máximo de plugins (MAX_PLUGINS) e ignora os extras.
• Parsing de arquivo inexistente: Verifica se o parser retorna erro negativo ao tentar ler um arquivo que não existe.
• Parsing de arquivo com linha malformada: Testa se o parser ignora linhas inválidas ou malformadas e processa apenas as válidas.
• Parsing de arquivo contendo apenas comentários e linhas vazias: Verifica se o parser retorna zero plugins quando o arquivo não contém configurações válidas.

Teste de driver

• Criação do driver com sucesso
  : Verifica se o driver é criado corretamente quando a alocação de memória e a inicialização do mutex funcionam.
• Falha na alocação de memória
  : Testa se o driver retorna NULL quando a função calloc falha.
• Falha na inicialização do mutex
  : Testa se o driver retorna NULL e libera a memória corretamente quando a inicialização do mutex falha.
• Manipulação da estrutura de dados do driver
  : Verifica se as informações dos plugins são armazenadas corretamente na estrutura do driver.
• Inicialização de plugin com falha
  : Testa o comportamento do driver quando a inicialização de um plugin falha (simulação simplificada).

[Copilot]

Pull Request Overview

Adds initial unit tests for plugin configuration parsing and plugin driver creation/initialization, introduces Ceedling/CMock configuration updates, and removes a Python header include from the driver interface. Key changes include: (1) new tests for config parsing edge cases and driver lifecycle paths, (2) support stubs and mockable headers for standard/pthread functions, (3) project.yml updated to enable mocks and Python/pthread linking while removing Python.h from the public driver header.

• Adds comprehensive parsing tests (valid file, over limit, malformed, empty, missing file).
• Adds driver creation and structural population tests plus a placeholder init failure test.
• Adjusts public header by dropping Python.h include (now only leaves PY_SSIZE_T_CLEAN macro).

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
tests/test_plugin_driver.c	Adds driver allocation, failure-path, structure population, and placeholder init tests plus direct function redefinitions.
tests/test_plugin_config.c	Adds multiple parsing behavior tests for plugin config files.
tests/support/test_plugin_driver_stubs.c	Provides stubs for external buffers and plugin_manager_destroy.
tests/support/mockable_stdlib.h	Declares mockable stdlib wrappers.
tests/support/mockable_pthread.h	Declares mockable pthread wrappers.
project.yml	Enables mocks, adds defines, Python/pthread libs, and test build configuration.
core/src/drivers/plugin_driver.h	Removes Python.h include but leaves PY_SSIZE_T_CLEAN macro.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

plugin_manager_destroy is already stubbed in tests/support/test_plugin_driver_stubs.c; this duplicate definition will cause a multiple definition error at link time. Remove one copy (prefer keeping it in the shared stub file).

Suggested change

	// Mock implementation for plugin_manager_destroy
	// This is normally defined in plcapp_manager.c
	void plugin_manager_destroy(PluginManager *manager)
	{
	(void)manager; // Suppress unused parameter warning
	// Mock implementation - do nothing
	}

[Copilot]

PY_SSIZE_T_CLEAN is intended to precede an inclusion of <Python.h>, but the header no longer includes Python.h. Either move both the macro and the include into the implementation (.c) file (if hiding the Python API is intended) or remove the macro here to avoid leaking a Python-specific macro into all includers.

[Copilot]

Overriding free without releasing the allocated memory causes intentional leaks that can accumulate across multiple tests and mask real lifetime issues. Consider invoking the real free (e.g., via wrapper or link-time wrapping) after incrementing the counter to avoid heap growth during the test suite.

[Copilot]

Expectation includes a trailing newline, while later similar assertion for plugin3 (line 81) omits it, creating inconsistent criteria that likely does not match intended parsing (newline should typically be stripped). Adjust to remove the newline (same issue also on line 73).

[Copilot]

[nitpick] Using magic numbers (0/1) for plugin types reduces readability; replace with the enum constants (e.g., PLUGIN_TYPE_PYTHON / PLUGIN_TYPE_NATIVE) for clarity and resilience to future enum reordering (also applies to lines 72 and 80).