Opened 4 years ago

Closed 3 years ago

#1220 closed enhancement (fixed)

more tests for PPIx::EditorTools

Reported by: bowtie Owned by:
Priority: minor Milestone:
Component: not classified yet Version: 0.85
Keywords: PPIx::EditorTools Cc:

Description

no Component for PPIx::EditorTools?

of note added a single test for PPI in 00-load.t, as Module::Install checks this, on install.

see attachment t.patch

as all changes are inside t

Attachments (3)

t.patch (14.6 KB) - added by bowtie 4 years ago.
Updated diff file as requested,
00-ppix-editortools.t (5.7 KB) - added by bowtie 4 years ago.
'spurious result' from abstract method?
t.patch.2 (17.1 KB) - added by bowtie 4 years ago.
Opps another go after ../tools/tidy_project.pl

Download all attachments as: .zip

Change History (7)

comment:1 Changed 4 years ago by bowtie

DC beforeTotal 90.5 49.4 48.4 99.4 52.6 100.0 79.0
Dc after t.parchTotal 93.1 50.7 48.7 99.4 52.6 100.0 81.5

better, but still not good enough.

DC points to PPIx::EditorTools
69.9 39.2 46.2 93.3 0.0 0.0 56.3

Ouch!!

Last edited 4 years ago by bowtie (previous) (diff)

Changed 4 years ago by bowtie

Updated diff file as requested,

comment:2 Changed 4 years ago by bowtie

Changed oo-load.t => 00-ppix-editortools.t

should we not also. set mime types correctly

  • *.t = svn:mine-type=text/x-perl

comment:3 Changed 4 years ago by bowtie

Guidance / Assistance required please.

with reference to the following section in 00-ppix-editortools.t

########
# Testing PPIx::EditorTools->find_unmatched_brace()
########

getting 'spurious result' from abstract method

Changed 4 years ago by bowtie

'spurious result' from abstract method?

Changed 4 years ago by bowtie

Opps another go after ../tools/tidy_project.pl

comment:4 Changed 3 years ago by bowtie

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.