[Bug 1763677] [NEW] Add all the sketches to the image for testing

classic Classic list List threaded Threaded
10 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] [NEW] Add all the sketches to the image for testing

Benoît St-Jean-5
Public bug reported:

It would be useful to add all the scripts/sketches (located at
https://bazaar.launchpad.net/~drgeo-
developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
image so we could run them into SUnit to detect regressions and/or use
them to profile Dr. Geo.

This is not a bug but a "nice to have" feature.

** Affects: drgeo
     Importance: Undecided
         Status: New


** Tags: have nice to wishlist

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  New

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Benoît St-Jean-5
I started adding all the sketches to the image.  Will provide a fileOut
of all classes soon. And will create a TestCase for every one of them as
well.

** Changed in: drgeo
       Status: New => In Progress

** Changed in: drgeo
     Assignee: (unassigned) => Benoît St-Jean (bstjean)

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Benoît St-Jean-5
In reply to this post by Benoît St-Jean-5
First version of the sketches.  Will do some minor cleanup in the
scripts (for instance, always naming the canvas with the name "canvas"
instead of "fig", "c", "sketch" or something else.

Will add test cases for each one later.

** Attachment added: "Sketches scripts added to the image"
   https://bugs.launchpad.net/drgeo/+bug/1763677/+attachment/5113781/+files/DrGeoII-Sketches.st

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Hliaire Fernandes
Administrator
In reply to this post by Benoît St-Jean-5
** Changed in: drgeo
   Importance: Undecided => Wishlist

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Hliaire Fernandes
Administrator
In reply to this post by Benoît St-Jean-5
** Changed in: drgeo
    Milestone: None => 18.06

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Hliaire Fernandes
Administrator
In reply to this post by Benoît St-Jean-5
Yep, using the Smalltalk sketches as the base for additional testings is
a great idea. Indeed, in the past the scripts helped to discover
additional bugs.

What about to have test cases where the resources are scripts load from
the file repository, then the test case just asserts it runs without
error?

That way, when one script is modified or added, a whole DrGeoII-Sketches
package doesn't need to be updated (or needed by the way).

What do you think about it?

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

Re: [Bug 1763677] Re: Add all the sketches to the image for testing

Benoît St-Jean-5
Yes, that would be way simpler!!

I was thinking about returning the canvas at the end of each script with
"^canvas" but after further thinking, all we need to do is trap any
error with an exception handler!


-----------------
Benoît St-Jean
Yahoo! Messenger: bstjean
Twitter: @BenLeChialeux
Pinterest: benoitstjean
Instagram: Chef_Benito
IRC: lamneth
Blogue: endormitoire.wordpress.com
"A standpoint is an intellectual horizon of radius zero".  (A. Einstein)

    On Friday, April 20, 2018, 3:31:08 a.m. EDT, hilaire <[hidden email]> wrote:  
 
 Yep, using the Smalltalk sketches as the base for additional testings is
a great idea. Indeed, in the past the scripts helped to discover
additional bugs.

What about to have test cases where the resources are scripts load from
the file repository, then the test case just asserts it runs without
error?

That way, when one script is modified or added, a whole DrGeoII-Sketches
package doesn't need to be updated (or needed by the way).

What do you think about it?

--
You received this bug notification because you are subscribed to the bug
report.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Hliaire Fernandes
Administrator
In reply to this post by Benoît St-Jean-5
** Tags removed: have nice to wishlist
** Tags added: script test

** Tags removed: script
** Tags added: smalltalk-sketch

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  In Progress

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Hliaire Fernandes
Administrator
In reply to this post by Benoît St-Jean-5
Added a DrGeoScriptTest class to test individually  each Smalltalk
sketch files

** Changed in: drgeo
       Status: In Progress => Fix Committed

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  Fix Committed

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1763677] Re: Add all the sketches to the image for testing

Hliaire Fernandes
Administrator
In reply to this post by Benoît St-Jean-5
** Changed in: drgeo
       Status: Fix Committed => Fix Released

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1763677

Title:
  Add all the sketches to the image for testing

Status in Dr. Geo II:
  Fix Released

Bug description:
  It would be useful to add all the scripts/sketches (located at
  https://bazaar.launchpad.net/~drgeo-
  developers/drgeo/trunk/files/head:/build/SmalltalkSketches/) to the
  image so we could run them into SUnit to detect regressions and/or use
  them to profile Dr. Geo.

  This is not a bug but a "nice to have" feature.

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1763677/+subscriptions