Skip to content

test(indexer): cover C++ module extensions#68

Merged
donk8r merged 1 commit into
Muvon:masterfrom
cnYui:codex/add-cpp-module-extension-detection
Jun 30, 2026
Merged

test(indexer): cover C++ module extensions#68
donk8r merged 1 commit into
Muvon:masterfrom
cnYui:codex/add-cpp-module-extension-detection

Conversation

@cnYui

@cnYui cnYui commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add C++ language regression tests for C++20 module file extensions.
  • Cover .cppm, .ixx, .mxx, .ccm, and .cxxm through Cpp::get_file_extensions, FileUtils::detect_language, and resolution_utils::detect_language_from_path.

Context

This adds test coverage for #50. Current master already recognizes these extensions; this PR locks the behavior down so module files keep being indexed as C++.

Verification

  • rustup run 1.95-x86_64-pc-windows-msvc cargo fmt --check
  • git diff --check

Attempted:

  • $env:PROTOC = .tmp-tools/protoc/bin/protoc.exe; rustup run 1.95-x86_64-pc-windows-msvc cargo test cpp_module_extensions

The test build progressed with rustc 1.95 and a local protoc, but native Windows linking failed because this machine's SDK/link environment is missing DXCORE.lib required by DirectML/ORT dependencies. The failure occurs at linker setup, not in the new C++ extension test assertions.

Refs #50

@donk8r donk8r merged commit c8036fa into Muvon:master Jun 30, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants