Totally up to suggestions.  :-)
Am 17.02.2013 15:54 schrieb "juliusv" <notifications@github.com>:

> In storage/metric/leveldb/regressions_test.go:
>
> >  )
> >
> > -func TestGetFingerprintsForLabelSetUsesAnd(t *testing.T) {
> > -	temporaryDirectory, _ := ioutil.TempDir("", "test_get_fingerprints_for_label_set_uses_and")
> > +var testGetFingerprintsForLabelSetUsesAndForLabelMatching = buildTestPersistence("get_fingerprints_for_labelset_uses_and_for_label_matching", metric.GetFingerprintsForLabelSetUsesAndForLabelMatchingTests)
>
> Hm, "buildTestPersistence" reads like you're only building a test
> persistence, not building *and testing* a persistence.
> I know the line is already crazy long, but something like
> "buildAndTestPersistence" would be clearer. Bleh... not sure.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/prometheus/prometheus/pull/79/files#r3042289>.
>
>