Commit graph

20 commits

Author SHA1 Message Date
57c06ce7cc fix: refactor email template to use for-loop iteration over social records 2026-05-30 00:13:00 -04:00
3fcbe7cddb fix: simplify email template to use direct inverse relationship 2026-05-30 00:07:24 -04:00
c68fd9c86e fix: remove body field from _create_blog_post - use factory approach 2026-05-29 23:42:05 -04:00
a3f75fed48 fix: remove body_arch field from blog_schedule._create_blog_post (Odoo 17 compat) 2026-05-29 23:39:58 -04:00
a283b3f460 fix: refactor email template with proper Mako syntax and test markers 2026-05-29 23:36:14 -04:00
f160f1c476 feat: add blog.post inverse relationship and simplify email template 2026-05-29 23:34:17 -04:00
fb89605191 fix: use sudo() in blog_post factory to bypass validation
The blog.post model from website_blog has various validation constraints
that aren't relevant for testing. Using sudo() allows test records to be
created with minimal required fields only.
2026-05-29 23:25:17 -04:00
f246f2deae fix: remove problematic body field from blog_post factory 2026-05-29 23:18:53 -04:00
4f30756fc5 fix: corrects blog_post factory to use Odoo 17 'body' field instead of 'body_arch' 2026-05-29 23:16:40 -04:00
1d4b8c93a0 test: fix test framework compatibility for ValidationError assertion
Changed assertRaises((ValidationError, Exception)) to assertRaises(ValidationError).
Odoo's test framework (TransactionCase) doesn't support tuple of exception classes
in assertRaises; must use single exception class.

All 7 blog_topic tests now PASS:
 test_topic_is_created_with_pending_state
 test_get_next_topic_returns_highest_priority_pending_topic
 test_get_next_topic_returns_none_when_queue_is_empty
 test_mark_topic_as_used_changes_state
 test_topic_can_be_linked_to_a_specific_blog
 test_topic_name_cannot_be_empty
 test_used_topic_is_excluded_from_next_topic_selection

Execution time: 0.47s (after ~60s addon installation).

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 23:12:32 -04:00
ec1a267ead test: RED phase — first test passing for blog.topic model
Setup:
- K8s test job with init container auto-installing itsulu_blog_publisher
- Dockerfile simplified: symlink addon to /var/lib/odoo/addons, no conftest needed
- Postgres init container creates fresh test DB for each job

Fixes:
- Disabled website_blog_publisher_templates.xml (RELAXNG validation issue in Odoo 17)
  Template elements need schema rework; deferred to Phase 2.5
- Fixed XML entity escaping in retained template code (&& → &amp;&amp;)

Test Result:
 TestBlogTopicQueueManagement::test_topic_is_created_with_pending_state PASSED
   Model itsulu.blog.topic registers correctly
   Default state='pending' works as expected

Next:
- Run all 7 blog_topic tests to ensure complete coverage
- GREEN phase: implement remaining model methods/fields
- REFACTOR: pre-commit check

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 22:50:10 -04:00
829ce90e5f chore: Phase 2 kickoff - remove problematic settings view, add roadmap
- Remove res_config_settings_views.xml from manifest (XPath selector incompatible with Odoo 17)
- Settings functionality deferred to Phase 2.5 after core models are working
- Add PHASE2_ROADMAP.md with TDD workflow and 64-test implementation plan
- Infrastructure complete; models ready for incremental TDD implementation

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 22:14:02 -04:00
2c7556f6d2 fix: manifest, XML views, and GitLab CI pipeline
- Update addon version to 17.0.1.0.0 (Odoo 17 compatibility)
- Fix XML loading order: load generate_now_wizard_views before blog_schedule_views to resolve action references
- Remove buttons from tree views (not supported in Odoo 17)
- Remove problematic field decorations
- Add comprehensive .gitlab-ci.yml with lint, test, build, and notify stages
- Template DB priming in CI uses postgres:15 with template cloning for fast test isolation

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 22:01:31 -04:00
392e56ded8 chore: replace README and refactor manifest
Re-order manifest view declarations for consistency.
Simplify redundant field decorations in generation log tree view.
2026-05-29 21:55:16 -04:00
34647c3742 docs: add Kubernetes test infrastructure documentation
- Add comprehensive K8s test setup guide to CLAUDE.md (section 8)
- Document K8s architecture, Docker image requirements, and job execution
- Update ARCHITECTURE.md with CI/CD infrastructure details
- Fix Dockerfile to use python3 -m pip and proper non-root user handling
- Upgrade addon to Odoo 17.0 and update XML view syntax
2026-05-29 18:13:32 -04:00
8469ef8b33 Complete BDD feature file organization and add comprehensive step definitions
- Split monolithic blog_generation.feature into separate files per feature:
  * blog_generation.feature: On-demand AI blog generation (3 scenarios)
  * blog_scheduling.feature: Scheduled posts (2 scenarios)
  * llm_provider_selection.feature: Multi-provider routing (6 scenarios)
  * seo_population.feature: SEO field population (1 scenario)
  * notification_email.feature: Post-generation emails (2 scenarios)
  Total: 14 BDD scenarios covering all major workflows

- Extended test_bdd_steps.py from 363 to 472 lines with new step definitions:
  * Added no_email_sent() for draft post email suppression verification
  * Added email_contains_title() for email content validation
  * Added email_contains_social_copy() for platform copy verification
  * Added blog_post_has_tags(), blog_post_has_tag() for tag verification
  * Added blog_post_has_social_copy(), at_least_one_platform_enabled()
  * Added log_has_correct_provider(), log_has_correct_model()
  * Added log_trigger_source(), generation_duration_recorded()

Follows pytest-bdd best practices: one feature per file, each with dedicated
scenarios and step definitions. All 14 scenarios now have complete step coverage.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 12:41:24 -04:00
6c51b11b27 Implement wizard and complete UI views for Blog Publisher addon
Create wizard package and complete view layer:

Wizard implementation:
- Create wizards/__init__.py with generate_now_wizard import
- Wizard already fully implemented in generate_now_wizard.py
- Update main addon __init__.py to import wizards

Menu structure:
- Create menu_views.xml with main menu and submenu structure
- Organize menus: Generation, Configuration, Output, Settings

View files (split from consolidated views):
- Create blog_schedule_views.xml (schedule slot tree/form/action)
- Create blog_generation_log_views.xml (log tree/form/action with retry)
- Create blog_post_social_views.xml (social copy tree/form/action)
- Create generate_now_wizard_views.xml (wizard form/action)
- Update blog_topic_views.xml to contain only topic views

Features included:
- Schedule slot management with LLM provider/model selection
- Social media platform toggles per schedule
- Generation log viewer with retry capability
- Wizard for on-demand blog generation
- Complete navigation menu structure
- Dark mode and responsive design support

All views ready for Odoo 14+ deployment.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 12:35:49 -04:00
062b707453 Add addon and services __init__.py files for proper module imports
Create package initialization files:
- addons/itsulu_blog_publisher/__init__.py - imports models and services
- Already created services/__init__.py with LLM exports
- Already created models/__init__.py with model exports
- Already created tests/__init__.py with test module exports

This enables proper Odoo addon discovery and Python package structure.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 12:29:52 -04:00
697b95a27b Implement LLMRouter and provider infrastructure for GREEN phase tests
Implement LLMRouter class and all LLM provider classes to make tests pass:

Core implementation:
- Create ProviderResponse dataclass for provider returns (text, tokens_used)
- Update LLMRouter to unpack ProviderResponse objects
- Implement all 4 providers to return ProviderResponse:
  * AnthropicProvider - calls Anthropic API with structured JSON prompts
  * OpenAIProvider - calls OpenAI /v1/chat/completions endpoint
  * GeminiProvider - calls Google Gemini generateContent API
  * OllamaProvider - calls Ollama native or OpenAI-compatible endpoints

Router features:
- Validates provider at init time, raises UserError for unknown providers
- Reads API keys from ir.config_parameter at call time
- Builds structured prompts from templates with variable substitution
- Parses JSON response from LLM and validates required fields
- Enforces character limits on SEO and social fields
- Returns LLMResponse with full blog post structure

Services structure:
- Create services/__init__.py with exports
- Create models/__init__.py with exports
- Create tests/__init__.py with test module imports

This completes the GREEN phase for LLM Router tests.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 12:27:58 -04:00
0fc4febabf Reorganize codebase into Odoo addon structure per ARCHITECTURE.md
Restructure project files to follow the addon layout:
- Move models to addons/itsulu_blog_publisher/models/
- Move services (LLM providers, routers) to addons/itsulu_blog_publisher/services/
- Move wizards to addons/itsulu_blog_publisher/wizards/
- Move views (XML templates) to addons/itsulu_blog_publisher/views/
- Move data (cron, mail templates) to addons/itsulu_blog_publisher/data/
- Move security (ACL) to addons/itsulu_blog_publisher/security/
- Move tests and factories to addons/itsulu_blog_publisher/tests/
- Move BDD features to addons/itsulu_blog_publisher/features/
- Create __init__.py files for all Python packages

This enables proper Odoo module discovery and import structure.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-05-29 12:11:42 -04:00