Fix missing CSS attr defaults

[kocaemre]

Summary

• Make CSS ::attr(...) values skip elements where the requested attribute is absent.
• Let .get(default) return the provided default for matched elements that lack the attribute.
• Add selector regression tests for present attributes, missing attributes, and getall().

Fixes #131.

Tests

• RED: python -m pytest tests/test_generic/test_selector.py::TestCss::test_css_attr_missing_attribute_uses_default tests/test_generic/test_selector.py::TestCss::test_css_attr_getall_skips_missing_attributes -q failed with '' == 'MISSING' and ['/present', ''] == ['/present']
• GREEN: python -m pytest tests/test_generic/test_selector.py -q → 21 passed
• GREEN: python -m pytest tests/ -q → 471 passed, 1 skipped, 19 deselected
• GREEN: ruff check src/ → All checks passed!

[Copilot]

Pull request overview

This PR updates SelectorList’s CSS ::attr(...) handling to align with Scrapy/Parsel-style semantics: elements that match the selector but lack the requested attribute no longer yield an empty-string value, allowing .get(default=...) to return the default and .getall() to exclude missing attributes.

Changes:

• Update SelectorList._values() for ::attr(...) to skip elements where the attribute key is absent.
• Add regression tests covering .get(default) for missing attributes and .getall() behavior when some elements lack the attribute.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/pyscrappy/generic/selector.py	Adjusts ::attr(...) value extraction to skip elements missing the attribute, enabling correct .get(default) semantics.
tests/test_generic/test_selector.py	Adds targeted tests for missing-attribute defaults and getall() skipping behavior.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.