Modify

Opened 12 months ago

Last modified 12 months ago

#23213 new enhancement

UI: Add buttons to enable/disable all validator tests

Reported by: gaben Owned by: team
Priority: minor Milestone:
Component: Core Version:
Keywords: template_report UI validator tests Cc:

Description

What steps will reproduce the problem?

  1. Develop a plugin which provides validator tests
  2. Try to profile the plugin validations on a large dataset
  3. The execution takes long time because of 40+ tests

What is the expected result?

UI has buttons which can enable and disable validator tests.

What happens instead?

I need to do it manually each time I clear JOSM preferences file.

Please provide any additional information below. Attach a screenshot if possible.

It would be mostly useful for plugin and core validator development.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2023-09-26 14:10:25 +0200 (Tue, 26 Sep 2023)
Build-Date:2023-09-27 01:31:02
Revision:18846
Relative:URL: ^/trunk

Attachments (1)

noTests.xml (7.9 KB ) - added by taylor.smock 12 months ago.

Download all attachments as: .zip

Change History (5)

comment:1 by taylor.smock, 12 months ago

Have you considered setting validator.skip and validator.skipBeforeUpload? I'll upload a preferences fragment that you can import from advanced preferences.

by taylor.smock, 12 months ago

Attachment: noTests.xml added

comment:2 by gaben, 12 months ago

Yeah I considered it but I was lazy to generate the list. And also it's changing as JOSM evolves, although it can do it for a while.

comment:3 by taylor.smock, 12 months ago

I'd rather have to generate/update the list. I don't want to make it easy for someone to accidentally disable all the validations.

comment:4 by gaben, 12 months ago

Originally I included in the report that it can show up if some cmd flag is present, like the --trace (? cannot recall which) for territories, but deleted it because I think it can be beneficial for normal users as well. At the same time, I get your point. Accidentally disabling all of the tests would make a mess.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to gaben.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.