Create unit tests for the PromptCodeBlockExtractor

Created on 19 November 2024, 10 months ago

Problem/Motivation

Currently there is no testing that the output of the PromptCodeBlockExtractor service is actually correct, we need to have extensive testing with multiple different strings being tested.

See the testing for PromptJsonDecoderTest for ideas.

Steps to reproduce

Proposed resolution

Add tests and test things like having ```html inside a text, having it start with
Also do the same for yaml, json, css
Test it with streaming

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Active

Version

1.0

Component

AI Core module

Created by

🇩🇪Germany marcus_johansson

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @marcus_johansson
  • 🇬🇧United Kingdom mjb3141
  • First commit to issue fork.
  • Merge request !321Resolve #3488376 "Create unit tests" → (Merged) created by Unnamed author
  • Pipeline finished with Failed
    9 months ago
    #362344
  • Pipeline finished with Failed
    9 months ago
    Total: 204s
    #362357
  • Pipeline finished with Failed
    9 months ago
    Total: 244s
    #362498
  • 🇩🇪Germany marcus_johansson

    Ah, I just realized these are kernel tests, but that is ok as well, we can merge these and set to fixed when Mark is finished with the unit tests as well (unless only kernel tests makes sense here).

  • 🇬🇧United Kingdom MrDaleSmith

    Tests currently failing.

  • Pipeline finished with Failed
    9 months ago
    Total: 164s
    #363515
  • Pipeline finished with Success
    9 months ago
    Total: 156s
    #363527
  • Pipeline finished with Success
    9 months ago
    Total: 157s
    #363555
  • Assigned to mjb3141
  • Status changed to Needs work 8 months ago
  • 🇩🇪Germany marcus_johansson

    It would still be good to move these out into external files before approve and merge.

  • Status changed to Fixed 5 days ago
Production build 0.71.5 2024