Opened 9 years ago

Closed 8 years ago

#12338 closed enhancement (duplicate)

fix the frobby optional tests to be marked "# optional - frobby" (very easy!)

Reported by: was Owned by: was
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: interfaces Keywords:
Cc: Merged in:
Authors: Reviewers: Miguel Marco
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

These tests should all be marked # optional - frobby, but none of them are:

sage: search_src('frobby')
misc/citation.pyx:26:systems['Frobby'] = ['sage.interfaces.frobby']
interfaces/    The official source for Frobby is \url{},
interfaces/                sage: frobby("analyze", input="vars x,y,z;[x^2,x*y];", #optional
interfaces/            We get an exception if frobby reports an error.
interfaces/                sage: frobby("do_dishes") #optional
interfaces/        command = ['frobby'] + action.split()
interfaces/                sage: decom = frobby.irreducible_decomposition(id) #optional
interfaces/                ...       decom0 = frobby.irreducible_decomposition(id0) #optional
interfaces/                ...       decom1 = frobby.irreducible_decomposition(id1) #optional
interfaces/        frobby_input = self._ideal_to_string(monomial_ideal)
interfaces/        frobby_output = self('irrdecom', input=frobby_input)
interfaces/        return self._parse_ideals(frobby_output, monomial_ideal.ring())
interfaces/            sage: parsed_ideals = frobby._parse_ideals(string, ring) #optional
interfaces/                sage: parsed_matrix = frobby._parse_4ti2_matrix(string) #optional
interfaces/                sage: frobby._parse_4ti2_matrix(string) #optional
interfaces/            sage: frobby._ideal_to_string(id) == "2 3\n2 0 0\n1 1 1\n" #optional
interfaces/            sage: frobby._monomial_to_string(monomial) == '2 0 1\n' #optional
interfaces/ = Frobby()
interfaces/ frobby import frobby

Once done, one should be able to test them with

 sage -t -only_optional=frobby

Fixed by #14841.

Change History (5)

comment:1 Changed 9 years ago by kcrisman

  • Description modified (diff)

However, not all tests actually always pass with the new package at #13007 - see the comment there. I'm not saying we should revert the review there, since it actually builds more places now, but at any rate probably we should fix more than just the frobby tests being marked this way - fixing them with the doctests themselves broken is pointless. So it's no longer "very easy", sadly.

comment:2 Changed 8 years ago by mmarco

I addressed this issue (among others) in #14841

comment:3 Changed 8 years ago by kcrisman

  • Milestone changed from sage-5.11 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

comment:4 Changed 8 years ago by kcrisman

  • Reviewers set to Miguel Marco
  • Status changed from needs_review to positive_review

comment:5 Changed 8 years ago by jdemeyer

  • Description modified (diff)
  • Resolution set to duplicate
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.