Valentina Cosentino

61 papers Journal 28Unranked 33
YearRankTypeTitle / Venue / Authors
2025 conf
MetroAutomotive
Giovanni Artale, Valentina Cosentino, Dario Costanzo, Antonio Delle Femine, Dario Di Cara, Vito Ditta, Daniele Gallo, Mario Luiso, Nicola Panzavecchia
2025 J jnl
IEEE Access
Nicola Panzavecchia, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Giovanni Tinè, Dario Di Cara
2025 conf
I2MTC
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Vito Ditta, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2025 J jnl
IEEE Trans. Instrum. Meas.
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Vito Ditta, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2025 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Antonio Delle Femine, Dario Di Cara, Vito Ditta, Daniele Gallo, Mario Luiso, Nicola Panzavecchia, Giovanni Tinè
2024 conf
AMPS
Salvatore Guaiana, Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Vito Ditta, Giovanni Tinè, Nicola Panzavecchia
2024 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Vito Ditta, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2023 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Lionel Cimaz, Valentina Cosentino, Dario Di Cara, Vito Ditta, Nunzio Dipaola, Salvatore Guaiana, Nicola Panzavecchia, Marilena Sambataro, Giovanni Tinè
2022 J jnl
IEEE Trans. Instrum. Meas.
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Riccardo Fiorelli, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2022 conf
I2MTC
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Vito Ditta, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2022 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Antonio Di Stefano, Nicola Panzavecchia, Giovanni Tinè, Aurelio Zinno
2021 conf
RTSI
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Vito Ditta, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2021 conf
AMPS
Giuseppe Caravello, Antonio Cataliotti, Dario Di Cara, Giovanni Artale, Valentina Cosentino, Giovanni Tinè, Vito Ditta, Aljaz Spelko, Bostjan Blazic
2021 conf
I2MTC
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Riccardo Fiorelli, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2021 J jnl
IEEE Trans. Instrum. Meas.
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2020 conf
MCSS
Giovanni Artale, Christof Brandauer, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Salvatore Guaiana, Nicola Panzavecchia, Stefano Salsano, Giovanni Tinè
2020 J jnl
IEEE Trans. Instrum. Meas.
Stefano Rinaldi, Marco Pasetti, Federico Bonafini, Paolo Ferrari, Alessandra Flammini, Emiliano Sisinni, Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè
2020 conf
I2MTC
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè
2020 J jnl
Sensors
Giovanni Artale, Giuseppe Caravello, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Nunzio Dipaola, Salvatore Guaiana, Nicola Panzavecchia, Marilena G. Sambataro, Giovanni Tinè
2019 conf
I2MTC
Giovanni Artale, Antonio Cataliotti, Giuseppe Caravello, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Vincenzo Antonucci, Marco Ferraro, Francesco Sergi
2019 conf
AMPS
Stefano Rinaldi, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Federico Bonafini, Paolo Ferrari, Alessandra Flammini, Marco Pasetti, Emiliano Sisinni, Giovanni Artale, Antonio Cataliotti, Valentina Cosentino
2019 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Cristiano Cervellera, Valentina Cosentino, Dario Di Cara, Mauro Gaggero, Danilo Macciò, Giuseppe Marsala, Antonella Ragusa, Giovanni Tinè
2019 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Gabriella Crotti, Antonio Delle Femine, Dario Di Cara, Daniele Gallo, Domenico Giordano, Carmine Landi, Mario Luiso, Mohammad Modarres, Giovanni Tinè
2018 J jnl
IEEE Trans. Smart Grid
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Riccardo Fiorelli, Salvatore Guaiana, Giovanni Tinè
2018 J jnl
IEEE Instrum. Meas. Mag.
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Riccardo Fiorelli, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2018 J jnl
IEEE Trans. Instrum. Meas.
Stefano Rinaldi, Federico Bonafini, Paolo Ferrari, Alessandra Flammini, Emiliano Sisinni, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana
2018 conf
RTSI
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Riccardo Fiorelli
2018 conf
I2MTC
Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Salvatore Nuccio, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè
2018 conf
I2MTC
Antonio Cataliotti, Valentina Cosentino, Gabriella Crotti, Domenico Giordano, Mohammad Modarres, Dario Di Cara, Giovanni Tinè, Daniele Gallo, Carmine Landi, Mario Luiso
2018 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Nunzio Dipaola, Marilena G. Sambataro
2017 conf
I2MTC
Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè
2017 J jnl
IEEE Trans. Instrum. Meas.
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Salvatore Nuccio, Giovanni Tinè
2017 conf
AMPS
Stefano Rinaldi, Federico Bonafini, Paolo Ferrari, Alessandra Flammini, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana
2016 conf
I2MTC
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio, Dario Di Cara, Giovanni Tinè, Giuseppe Privitera
2016 J jnl
IEEE Trans. Smart Grid
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Pierluca Russotto, Enrico Telaretti, Giovanni Tinè
2016 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Giovanni Tinè
2016 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Giovanni Tinè
2015 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè
2015 conf
AMPS
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Salvatore Guaiana, Nicola Panzavecchia, Giovanni Tinè, Daniele Gallo, Carmine Landi, M. Landi, Mario Luiso
2015 conf
I2MTC
Antonio Cataliotti, Valentina Cosentino, Enrico Telaretti, Dario Di Cara, Giovanni Tinè
2014 conf
AMPS
Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè
2014 conf
I2MTC
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Giuseppe Privitera
2014 conf
I2MTC
Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè
2014 conf
SmartGridComm
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Salvatore Guaiana, Dario Di Cara, Nicola Panzavecchia, Giovanni Tinè, Riccardo Fiorelli
2013 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Giovanni Tinè
2013 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Ngoctrung Nguyen, Dario Di Cara, Giovanni Tinè
2013 J jnl
IEEE Trans. Instrum. Meas.
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Riccardo Fiorelli, Pierluca Russotto, Giovanni Tinè
2012 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Alessandro Lipari, Salvatore Nuccio, Ciro Spataro
2012 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Giovanni Tinè
2012 conf
AMPS
Giovanni Artale, Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Pierluca Russotto, Riccardo Fiorelli, Giovanni Tinè
2011 conf
AMPS
Antonio Cataliotti, Valentina Cosentino, Dario Di Cara, Giovanni Tinè
2010 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino
2010 conf
AMPS
Antonio Cataliotti, Valentina Cosentino
2009 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino
2009 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Alessandro Lipari, Salvatore Nuccio
2009 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio
2008 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio
2008 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio
2007 J jnl
IEEE Trans. Instrum. Meas.
Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio
2007 J jnl
IEEE Trans. Instrum. Meas.
Massimo Aiello, Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio
2005 J jnl
IEEE Trans. Instrum. Meas.
Massimo Aiello, Antonio Cataliotti, Valentina Cosentino, Salvatore Nuccio
tests/integration/test_pe_extractors.py
← Index tests/integration/test_pe_extractors.py python
"""
Integration tests for PE-specific extractors.
"""
import pytest
from unittest.mock import Mock, patch, MagicMock

pytestmark = [pytest.mark.integration, pytest.mark.pe]


# ============================================================================
# PEFeaturesExtractor Tests
# ============================================================================

class TestPEFeaturesExtractor:
    """Tests for PEFeaturesExtractor class."""

    def test_extract_valid_pe(self, pe_binary_path, mock_logger):
        """Test extracting features from valid PE file."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor
        from redb.models.dataclasses import PE

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            assert result is not None
            assert isinstance(result, PE)

    def test_extract_pe_type(self, pe_binary_path, mock_logger):
        """Test PE type detection (DLL, EXE, DRIVER)."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            assert result.type in ["DLL", "EXE", "DRIVER"]

    def test_extract_architecture(self, pe_binary_path, mock_logger):
        """Test architecture detection."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # Architecture should be detected
            assert result.architecture is not None

    def test_extract_entry_point(self, pe_binary_path, mock_logger):
        """Test entry point extraction."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # Entry point should be a hex string
            assert result.entry_point.startswith("0x")

    def test_extract_compilation_time(self, pe_binary_path, mock_logger):
        """Test compilation time extraction."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # Compilation time should be an integer timestamp
            assert isinstance(result.compilation_time, int)
            assert result.compilation_time_utc is not None

    def test_extract_dotnet_detection(self, pe_binary_path, mock_logger):
        """Test .NET detection."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # Should be True for the test .NET binary
            assert isinstance(result.is_dotnet, bool)
            assert result.is_dotnet is True  # Test file is .NET

    def test_extract_headers(self, pe_binary_path, mock_logger):
        """Test header extraction."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            assert result.dos_header is not None
            assert result.nt_header is not None
            assert result.file_header is not None
            assert result.optional_header is not None

    def test_extract_counts(self, pe_binary_path, mock_logger):
        """Test section/import/export counts."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            assert isinstance(result.number_of_sections, int)
            assert isinstance(result.number_of_imports, int)
            assert isinstance(result.number_of_exports, int)
            assert isinstance(result.number_of_resources, int)

    def test_extract_rich_header(self, pe_binary_path, mock_logger):
        """Test Rich header extraction."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # Rich header may or may not be present
            # If present, should be JSON string
            if result.rich_header_dump is not None:
                assert isinstance(result.rich_header_dump, str)

    def test_extract_version_info(self, pe_binary_path, mock_logger):
        """Test version info extraction."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEFeaturesExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # Version info may or may not be present
            if result.version_info is not None:
                assert isinstance(result.version_info, list)


# ============================================================================
# PEImportExtractor Tests
# ============================================================================

class TestPEImportExtractor:
    """Tests for PEImportExtractor class."""

    def test_extract_imports(self, pe_binary_path, mock_logger):
        """Test import extraction."""
        from redb.extractors.pe_extractors.pe_imports import PEImportExtractor
        from redb.models.dataclasses import PEImport

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEImportExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            if result is not None:
                assert isinstance(result, PEImport)
                assert isinstance(result.pe_imports_total, int)

    def test_extract_import_libraries(self, pe_binary_path, mock_logger):
        """Test import library extraction."""
        from redb.extractors.pe_extractors.pe_imports import PEImportExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEImportExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            if result is not None and result.pe_import_libraryName is not None:
                assert isinstance(result.pe_import_libraryName, list)

    def test_extract_import_functions(self, pe_binary_path, mock_logger):
        """Test import function extraction."""
        from redb.extractors.pe_extractors.pe_imports import PEImportExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEImportExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            if result is not None and result.pe_import_functions is not None:
                assert isinstance(result.pe_import_functions, list)

    def test_prepare_export_clickhouse(self, pe_binary_path, mock_logger):
        """Test ClickHouse export preparation."""
        from redb.extractors.pe_extractors.pe_imports import PEImportExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEImportExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.prepare_export_data("ClickHouseExporter")

            if result is not None:
                data, column_names, column_type_names = result
                assert 'library_name' in column_names
                assert 'function_name' in column_names


# ============================================================================
# PESectionExtractor Tests
# ============================================================================

class TestPESectionExtractor:
    """Tests for PESectionExtractor class."""

    def test_extract_sections(self, pe_binary_path, mock_logger):
        """Test section extraction."""
        from redb.extractors.pe_extractors.pe_sections import PESectionExtractor
        from redb.models.dataclasses import PESection

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PESectionExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            assert result is not None
            assert isinstance(result, list)
            assert len(result) > 0
            assert isinstance(result[0], PESection)

    def test_section_properties(self, pe_binary_path, mock_logger):
        """Test section property extraction."""
        from redb.extractors.pe_extractors.pe_sections import PESectionExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PESectionExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            if result is not None and len(result) > 0:
                section = result[0]
                assert section.section_name is not None
                assert section.section_entropy >= 0
                assert section.section_sha256 is not None
                assert section.section_md5 is not None
                assert section.section_size >= 0

    def test_section_entropy_range(self, pe_binary_path, mock_logger):
        """Test that section entropy is in valid range."""
        from redb.extractors.pe_extractors.pe_sections import PESectionExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PESectionExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            if result is not None:
                for section in result:
                    assert 0 <= section.section_entropy <= 8

    def test_prepare_export_clickhouse(self, pe_binary_path, mock_logger):
        """Test ClickHouse export preparation."""
        from redb.extractors.pe_extractors.pe_sections import PESectionExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PESectionExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.prepare_export_data("ClickHouseExporter")

            if result is not None:
                data, column_names, column_type_names = result
                assert 'section_name' in column_names
                assert 'section_entropy' in column_names
                assert 'section_sha256' in column_names


# ============================================================================
# PEResourceExtractor Tests
# ============================================================================

class TestPEResourceExtractor:
    """Tests for PEResourceExtractor class."""

    def test_extract_resources(self, pe_binary_path, mock_logger):
        """Test resource extraction."""
        from redb.extractors.pe_extractors.pe_resources import PEResourceExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEResourceExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # May be None if no resources
            if result is not None:
                assert isinstance(result, list)

    def test_resource_properties(self, pe_binary_path, mock_logger):
        """Test resource property extraction."""
        from redb.extractors.pe_extractors.pe_resources import PEResourceExtractor
        from redb.models.dataclasses import PEResource

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEResourceExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            if result is not None and len(result) > 0:
                resource = result[0]
                assert isinstance(resource, PEResource)
                assert resource.resource_type is not None


# ============================================================================
# PEOverlayExtractor Tests
# ============================================================================

class TestPEOverlayExtractor:
    """Tests for PEOverlayExtractor class."""

    def test_extract_overlay(self, pe_binary_path, mock_logger):
        """Test overlay extraction."""
        from redb.extractors.pe_extractors.pe_overlay import PEOverlayExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEOverlayExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # May be None if no overlay
            # If present, should have overlay properties
            if result is not None:
                assert hasattr(result, 'overlay_size')

    def test_has_overlay_check(self, pe_binary_path, mock_logger):
        """Test overlay detection."""
        from redb.extractors.pe_extractors.pe_overlay import PEOverlayExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEOverlayExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            has_overlay = extractor._has_overlay()
            assert isinstance(has_overlay, bool)


# ============================================================================
# PESignatureExtractor Tests
# ============================================================================

class TestPESignatureExtractor:
    """Tests for PESignatureExtractor class."""

    def test_extract_signature(self, pe_binary_path, mock_logger):
        """Test signature extraction."""
        from redb.extractors.pe_extractors.pe_signature import PESignatureExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PESignatureExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # May be None if not signed
            # Result type depends on implementation

    def test_is_signed_check(self, pe_binary_path, mock_logger):
        """Test signature detection."""
        from redb.extractors.pe_extractors.pe_signature import PESignatureExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PESignatureExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            is_signed = extractor._is_signed()
            assert isinstance(is_signed, bool)


# ============================================================================
# PEDotNetExtractor Tests
# ============================================================================

class TestPEDotNetExtractor:
    """Tests for PEDotNetExtractor class."""

    def test_extract_dotnet(self, pe_binary_path, mock_logger):
        """Test .NET metadata extraction."""
        from redb.extractors.pe_extractors.pe_dotnet import PEDotNetExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEDotNetExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # The test file is a .NET binary
            if extractor._check_dotnet():
                assert result is not None

    def test_check_dotnet(self, pe_binary_path, mock_logger):
        """Test .NET detection."""
        from redb.extractors.pe_extractors.pe_dotnet import PEDotNetExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEDotNetExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            is_dotnet = extractor._check_dotnet()
            assert isinstance(is_dotnet, bool)
            # Test file should be .NET
            assert is_dotnet is True


# ============================================================================
# PEInconsistencyTestsExtractor Tests
# ============================================================================

class TestPEInconsistencyTestsExtractor:
    """Tests for PEInconsistencyTestsExtractor class."""

    def test_extract_inconsistency_tests(self, pe_binary_path, mock_logger):
        """Test inconsistency tests extraction."""
        from redb.extractors.pe_extractors.pe_inconsistency_tests import PEInconstistencyTestsExtractor as PEInconsistencyTestsExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEInconsistencyTestsExtractor(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # extract() returns True if tests were performed, False otherwise
            # The actual test results are stored in extractor.pe_inconsistency_tests
            # and extractor.dotnet_inconsistency_tests
            assert result in (True, False)

            if result:
                # Check that test results were stored
                assert extractor.pe_inconsistency_tests is not None or extractor.dotnet_inconsistency_tests is not None


# ============================================================================
# PEExtraFindings Tests
# ============================================================================

class TestPEExtraFindings:
    """Tests for PEExtraFindings class."""

    def test_extract_extra_findings(self, pe_binary_path, mock_logger):
        """Test extra findings extraction."""
        from redb.extractors.pe_extractors.pe_extra_findings import PEExtraFindings

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            extractor = PEExtraFindings(pe_binary_path, mock_logger)

            if extractor.pe is None:
                pytest.skip("PE parsing failed")

            result = extractor.extract()

            # May return None or list of findings
            if result is not None:
                assert isinstance(result, list)


# ============================================================================
# Integration Tests
# ============================================================================

class TestPEExtractorIntegration:
    """Integration tests for PE extractors."""

    def test_all_extractors_same_pe_object(self, pe_binary_path, mock_logger, pe_object):
        """Test that extractors can share the same PE object."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor
        from redb.extractors.pe_extractors.pe_imports import PEImportExtractor
        from redb.extractors.pe_extractors.pe_sections import PESectionExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            # Create extractors sharing the same PE object
            features_ext = PEFeaturesExtractor(pe_binary_path, mock_logger, pe=pe_object)
            imports_ext = PEImportExtractor(pe_binary_path, mock_logger, pe=pe_object)
            sections_ext = PESectionExtractor(pe_binary_path, mock_logger, pe=pe_object)

            # All should use the same PE object
            assert features_ext.pe is pe_object
            assert imports_ext.pe is pe_object
            assert sections_ext.pe is pe_object

            # All should extract successfully
            features_result = features_ext.extract()
            imports_result = imports_ext.extract()
            sections_result = sections_ext.extract()

            assert features_result is not None
            # imports and sections may be None if not present

    def test_hash_consistency_across_extractors(self, pe_binary_path, mock_logger):
        """Test that hashes are consistent across all extractors."""
        from redb.extractors.pe_extractors.pe_features import PEFeaturesExtractor
        from redb.extractors.pe_extractors.pe_imports import PEImportExtractor

        with patch('redb.settings.ELASTIC_BINARIES_COLLECTION', 'test'):
            features_ext = PEFeaturesExtractor(pe_binary_path, mock_logger)
            imports_ext = PEImportExtractor(pe_binary_path, mock_logger)

            # Hashes should be the same
            assert features_ext.sha256 == imports_ext.sha256
            assert features_ext.md5 == imports_ext.md5
            assert features_ext.sha1 == imports_ext.sha1