29
44%
master: 98%

Ran 04 Dec 2019 05:42PM UTC

Jobs 1

Files 75

Run time 5s

Badge

Embed ▾

pending completion

Build # 29

Build Type

Pull #8

travis-ci

Committed by José Joaquín Atria

Commit Message

Make sure ->app always holds a (weak) reference to the app

App introspection is possible by holding a reference to the app in each
of its commands. In the past, this was the package name of the app's
package, but this made it impossible to have eg. multiple instances of
an app, or to have an app whose state is not entirely known at compile
time.

This patch solves this by holding a reference instead, which is made
weak to reduce the risk of memory leaks. For consistency (and since
a common pattern seems to be for apps to subclass both App::CLI
and App::CLI::Command), the app also gets a weak reference to itself.

An additional change in cmd_map makes sure the package name is properly
constructed.

Pull Request Pull Request #8: Make sure ->app always holds a (weak) reference to the app

Run Details

1588 of 3604 relevant lines covered (44.06%)

0.44 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
9	29.9 (RELEASE_TESTING=1)	04 Dec 2019 05:42PM UTC	0	44.06	Travis Job 29.9

Source Files on build 29

Detailed source file information is not available for this build.

paultcochrane / app-cli / 29
44%
master: 98%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 29

paultcochrane / app-cli / 29 44% master: 98%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 29

paultcochrane / app-cli / 29
44%
master: 98%

README BADGES
x